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

[Phase 1][YSQL][Colocation] ALTER TABLE ADD CONSTRAINT does not support [NO] TABLEGROUP. #11600

Closed
frozenspider opened this issue Feb 25, 2022 · 2 comments
Closed

[Phase 1][YSQL][Colocation] ALTER TABLE ADD CONSTRAINT does not support [NO] TABLEGROUP. #11600

frozenspider opened this issue Feb 25, 2022 · 2 comments
Assignees
@m-iancu
Labels
area/ysql Yugabyte SQL (YSQL) kind/bug This issue is a bug priority/medium Medium priority issue wontfix

Comments

@frozenspider
Copy link
Contributor

frozenspider commented Feb 25, 2022 •
edited by jira bot
Loading

Jira Link: [DB-373](https://yugabyte.atlassian.net/browse/DB-373)

Description

However, you can still CREATE INDEX ... NO TABLEGROUP (or TABLEGROUP x) followed by ALTER TABLE ADD UNIQUE USING INDEX, in which case NO TABLEGROUP will be lost during ysql_dump --include-yb-metadata which will break index when carrying over SST files.
@frozenspider frozenspider added the area/ysql Yugabyte SQL (YSQL) label Feb 25, 2022
@frozenspider frozenspider changed the title [YSQL][Colocation] ALTER TABLE ADD CONSTRAINT does not support NO TABLEGROUP. [YSQL][Colocation] ALTER TABLE ADD CONSTRAINT does not support [NO] TABLEGROUP. Feb 25, 2022
@frozenspider frozenspider mentioned this issue Mar 9, 2022
Open
@m-iancu
Copy link
Contributor

m-iancu commented Apr 20, 2022

Different from but likely related to: #11583 -- the fix should probably follow the same idea for both.

@tverona1 tverona1 changed the title [YSQL][Colocation] ALTER TABLE ADD CONSTRAINT does not support [NO] TABLEGROUP. [Phase 1] [YSQL][Colocation] ALTER TABLE ADD CONSTRAINT does not support [NO] TABLEGROUP. May 17, 2022
@yugabyte-ci yugabyte-ci added kind/bug This issue is a bug priority/medium Medium priority issue labels May 17, 2022
@frozenspider frozenspider changed the title [Phase 1] [YSQL][Colocation] ALTER TABLE ADD CONSTRAINT does not support [NO] TABLEGROUP. [Phase 1][YSQL][Colocation] ALTER TABLE ADD CONSTRAINT does not support [NO] TABLEGROUP. May 18, 2022
@frozenspider
Copy link
Contributor Author

We've decided to restrict table/index to share a tablegroup, so this becomes outdated.
Closed in favour of #12503

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

@m-iancu m-iancu

Labels
area/ysql Yugabyte SQL (YSQL) kind/bug This issue is a bug priority/medium Medium priority issue wontfix
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@frozenspider @m-iancu @yugabyte-ci