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

geo: Error when loading from an older version of sys catalog #11509

Closed
es1024 opened this issue Feb 17, 2022 · 0 comments
Closed

geo: Error when loading from an older version of sys catalog #11509

es1024 opened this issue Feb 17, 2022 · 0 comments
Assignees
@es1024
Labels
area/docdb YugabyteDB core features kind/bug This issue is a bug
Projects
Row Level Geo Partitioning

Comments

@es1024
Copy link
Contributor

es1024 commented Feb 17, 2022 •
edited

Description

50941ff added an entry in sys catalog to keep track of transaction status tables version, but this entry does not exist in older versions of sys catalog, causing fatal errors. This entry should be added in after catalog loading if it did not get set during load.

Context: #9980, #11402
@es1024 es1024 added kind/bug This issue is a bug area/docdb YugabyteDB core features labels Feb 17, 2022
@es1024 es1024 self-assigned this Feb 17, 2022
@es1024 es1024 added this to To Do in Row Level Geo Partitioning via automation Feb 17, 2022
es1024 added a commit that referenced this issue Feb 18, 2022
@es1024
[#11509] geo: Fix error when loading from an older version of sys cat…
5e62c55 
…alog

Summary:
Added initialization code for transaction_tables_config_ after loading sys catalog if it
was not set during loading.

Test Plan:
Ran reinitdb before D15388 / 50941ff, built master,
verified error from CHECK_NOTNULL, built with changes without rerunning reinitdb, verified that
tests now pass.

Reviewers: sergei

Reviewed By: sergei

Subscribers: rthallam, bogdan, ybase

Differential Revision: https://phabricator.dev.yugabyte.com/D15537
@es1024 es1024 closed this as completed Feb 18, 2022
Row Level Geo Partitioning automation moved this from To Do to Done Feb 18, 2022
jayant07-yb pushed a commit to jayant07-yb/yugabyte-db that referenced this issue Mar 8, 2022
@es1024
[yugabyte#11509] geo: Fix error when loading from an older version of…
13f464b 
… sys catalog

Summary:
Added initialization code for transaction_tables_config_ after loading sys catalog if it
was not set during loading.

Test Plan:
Ran reinitdb before D15388 / 50941ff, built master,
verified error from CHECK_NOTNULL, built with changes without rerunning reinitdb, verified that
tests now pass.

Reviewers: sergei

Reviewed By: sergei

Subscribers: rthallam, bogdan, ybase

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

@es1024 es1024

Labels
area/docdb YugabyteDB core features kind/bug This issue is a bug
Projects
Row Level Geo Partitioning
Done
Milestone
No milestone
Development

No branches or pull requests
1 participant
@es1024