Skip to content

Update the changing database default tablespace#558

Open
Andriciuc wants to merge 2 commits intomainfrom
doc-alter-set-issue
Open

Update the changing database default tablespace#558
Andriciuc wants to merge 2 commits intomainfrom
doc-alter-set-issue

Conversation

@Andriciuc
Copy link
Copy Markdown
Collaborator

Update the database default tablespace behavior description to include alter/set command

Previously the explanation was not quite helpful, as in the user can technically still use the command, previously we said it was not supported which is not true.

Update the database default tablespace behavior description to include alter/set command

Previously the explanation was not quite helpful, as in the user can technically still use the command, previously we said it was not supported which is not true.
As a safeguard, `pg_tde` blocks the operation if encrypted objects are detected in the default tablespace.

Objects located outside the default tablespace are not affected by this command.
Only objects in the default tablespace are checked. Objects in other tablespaces are not affected by this validation.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should add the suggestion that you can move the encrypted tables one by one.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated, added that users can use ALTER TABLE SET TABLESPACE to move tables individually, should we also suggest the ALTER INDEX SET TABLESPACE command?

Inform user they can use ALTER table to move tables individually.
Only objects in the default tablespace are checked. Objects in other tablespaces are not evaluated by `pg_tde`.

Objects located outside the default tablespace are not affected by this command.
To move encrypted tables individually, use `ALTER TABLE ... SET TABLESPACE`.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A bit weird way to phrase it. Maybe it should be more like: "A workaround is to ...".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants