Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[YSQL] ALTER SCHEMA not supported #6470

Closed
ajcaldera1 opened this issue Nov 24, 2020 · 0 comments
Closed

[YSQL] ALTER SCHEMA not supported #6470

ajcaldera1 opened this issue Nov 24, 2020 · 0 comments
Assignees
@d-uspenskiy
Labels
area/ysql Yugabyte SQL (YSQL)
Milestone
2.7.x

Comments

@ajcaldera1
Copy link
Contributor

As a user, I would like to be able to rename a schema (namespace) once it has been created. Postgres supports this via

ALTER SCHEMA foo RENAME TO blah;
@ajcaldera1 ajcaldera1 added the area/ysql Yugabyte SQL (YSQL) label Nov 24, 2020
@m-iancu m-iancu added this to the 2.7.x milestone May 26, 2021
@m-iancu m-iancu assigned d-uspenskiy and unassigned m-iancu Jul 23, 2021
d-uspenskiy added a commit that referenced this issue Jul 28, 2021
@d-uspenskiy
[#6470] YSQL: Enable ALTER SCHEMA RENAME
39b2b6b 
Summary:
DocDB layer knows nothing about postgres schemas. This information in stored only in postgres sys catalog.
It is enough to allow the `ALTER SCHEMA RENAME` command on parser level to support schema reaming.

Test Plan:
postgres's regress test `namespace.sql` was integrated as `yb_pg_namespace.sql`

```
./yb_build.sh --java-test 'org.yb.pgsql.TestPgRegressPgMiscIndependent'
```

Reviewers: alex, mihnea

Reviewed By: mihnea

Subscribers: yql

Differential Revision: https://phabricator.dev.yugabyte.com/D12387
d-uspenskiy added a commit that referenced this issue Aug 6, 2021
@d-uspenskiy
[BACKPORT 2.6][#6470] YSQL: Enable ALTER SCHEMA RENAME
d6ad691 
Summary:
DocDB layer knows nothing about postgres schemas. This information in stored only in postgres sys catalog.
It is enough to allow the `ALTER SCHEMA RENAME` command on parser level to support schema reaming.

Original commit: D12387 / 39b2b6b

Test Plan:
Jenkins: rebase: 2.6
postgres's regress test `namespace.sql` was integrated as `yb_pg_namespace.sql`

```
./yb_build.sh --java-test 'org.yb.pgsql.TestPgRegressPgMiscIndependent'
```

Reviewers: alex, mihnea

Reviewed By: mihnea

Subscribers: yql

Differential Revision: https://phabricator.dev.yugabyte.com/D12471
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@d-uspenskiy d-uspenskiy

Labels
area/ysql Yugabyte SQL (YSQL)
Projects
None yet
Milestone
2.7.x
Development

No branches or pull requests
3 participants
@m-iancu @ajcaldera1 @d-uspenskiy