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.

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

[DocDB] Check failed during catalog loading: tablegroup already exists #14104

Closed
msun07 opened this issue Sep 20, 2022 · 0 comments
Closed

[DocDB] Check failed during catalog loading: tablegroup already exists #14104

msun07 opened this issue Sep 20, 2022 · 0 comments
Assignees
Labels
area/docdb YugabyteDB core features kind/bug This issue is a bug priority/medium Medium priority issue

Comments

@msun07
Copy link
Contributor

msun07 commented Sep 20, 2022

Jira Link: DB-3591

Description

  1. Create a tablegroup with two child tables in rf=3 cluster
  2. Change the leader to another node
  3. Change back the leader role
ca970df2ab154becb3d51653f65b1163: Load: Loading tablets into memory.
I0920 17:41:04.446362 1897328640 catalog_manager.cc:1235] T 00000000000000000000000000000000 P ca970df2ab154becb3d51653f65b1163: Load: Loading tablets into memory.
I0920 17:41:04.450559 1897328640 catalog_loaders.cc:313] Loaded metadata for tablet 00000000000000000000000000000000 (first table sys.catalog [id=sys.catalog.uuid])
I0920 17:41:04.450697 1897328640 catalog_loaders.cc:313] Loaded metadata for tablet 0765ef4cc29f4fd0b9f723ec167cb8b3 (first table keyspaces [id=0eb68f27530e47e6aaae3417147ca665])
I0920 17:41:04.450788 1897328640 catalog_loaders.cc:313] Loaded metadata for tablet 07c92d353afa455791d0d43dc93a7668 (first table transactions [id=57c3773a4b9c433b8fba293b4dd85460])
I0920 17:41:04.450855 1897328640 catalog_loaders.cc:313] Loaded metadata for tablet 08aeac44457841b9a8723f59667aea97 (first table roles [id=08e30513b14b431cbb80265f1b8a5170])
I0920 17:41:04.450935 1897328640 catalog_loaders.cc:313] Loaded metadata for tablet 08f1766accf346128dd1772973b7f047 (first table transactions [id=57c3773a4b9c433b8fba293b4dd85460])
... ...
F0920 17:41:04.453138 1897328640 ysql_tablegroup_manager.cc:80] Check failed: !ContainsKey(tablegroup_map_, tablegroup_id) Tablegroup 00004000000030008000000000004001 already exists
@msun07 msun07 added area/docdb YugabyteDB core features status/awaiting-triage Issue awaiting triage labels Sep 20, 2022
@yugabyte-ci yugabyte-ci added kind/bug This issue is a bug priority/medium Medium priority issue labels Sep 20, 2022
@msun07 msun07 self-assigned this Sep 20, 2022
@yugabyte-ci yugabyte-ci removed the status/awaiting-triage Issue awaiting triage label Sep 20, 2022
msun07 added a commit that referenced this issue Sep 23, 2022
Summary:
The diff adds PITR support for tablegroups. In particular, dropping the tablegroup parent tablet is supported.
It adds tablegroup GC tests and parameterizes the existing test cases to verify Ysql tablegroup PITR.

Check failed during catalog loading also gets fixed by adding tablegroup map cleanup logic.

Test Plan:
ybd --cxx-test yb-admin-snapshot-schedule-test

ybd --cxx-test snapshot-schedule-test --gtest_filter SnapshotScheduleTest.TablegroupGC
ybd --cxx-test yb-admin-snapshot-schedule-test --gtest_filter Colocation/YbAdminSnapshotScheduleTestWithYsqlParam.*

Reviewers: alex, zdrudi, skedia

Reviewed By: alex, zdrudi, skedia

Subscribers: ybase, bogdan

Differential Revision: https://phabricator.dev.yugabyte.com/D19398
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/docdb YugabyteDB core features kind/bug This issue is a bug priority/medium Medium priority issue
Projects
None yet
Development

No branches or pull requests

2 participants