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: Local transaction status tablets not loaded into transaction cache on TS restart #10828

Closed
es1024 opened this issue Dec 10, 2021 · 1 comment
Closed

geo: Local transaction status tablets not loaded into transaction cache on TS restart #10828

es1024 opened this issue Dec 10, 2021 · 1 comment
Assignees
@es1024
Labels
area/docdb YugabyteDB core features
Projects
Row Level Geo Partitioning

Comments

@es1024
Copy link
Contributor

es1024 commented Dec 10, 2021 •
edited

Description

After TS restart, transaction status tablets are fetched before RemoteTabletServer has its placement set, resulting in the initial fetch always only fetching global tablets. No further fetch is done until a change in transaction status tables happens (new transaction status table created/placement set), until which all queries through the restarted TS will always use the global transaction table.

Context: #10158, #9980
@es1024 es1024 added the area/docdb YugabyteDB core features label Dec 10, 2021
@es1024 es1024 self-assigned this Dec 10, 2021
@es1024 es1024 added this to To Do in Row Level Geo Partitioning via automation Dec 10, 2021
es1024 added a commit that referenced this issue Jan 27, 2022
@es1024
[#10536, #10828] geo: Consolidate TS txn status tablet cache update i…
0c05d05 
…nto one RPC

Summary:
Consolidates the many RPCs made when the transaction manager updates the
transaction status tablet cache into a single RPC which returns the relevant lists
of global and local transaction status tablet ids. Addresses bug where the initial
fetch of transaction table tablets in TransactionManager does not fetch placement
local tablets.

Test Plan: `ybd --cxx-test pgwrapper_geo_transactions-test`

Reviewers: bogdan, mbautin, sergei

Reviewed By: sergei

Subscribers: ybase

Differential Revision: https://phabricator.dev.yugabyte.com/D13957
@es1024
Copy link
Contributor Author

es1024 commented Jan 27, 2022

Resolved by 0c05d05

@es1024 es1024 closed this as completed Jan 27, 2022
Row Level Geo Partitioning automation moved this from To Do to Done Jan 27, 2022
es1024 added a commit that referenced this issue Feb 2, 2022
@es1024
[Backport 2.12] [#10536, #10828] geo: Consolidate TS txn status table…
8f12f90 
…t cache update into one RPC

Summary:
Consolidates the many RPCs made when the transaction manager updates the
transaction status tablet cache into a single RPC which returns the relevant lists
of global and local transaction status tablet ids. Addresses bug where the initial
fetch of transaction table tablets in TransactionManager does not fetch placement
local tablets.

Original Commit: D13957 / 0c05d05

Test Plan:
Jenkins: rebase 2.12, urgent

`ybd --cxx-test pgwrapper_geo_transactions-test`

Reviewers: bogdan, mbautin, sergei

Subscribers: ybase

Differential Revision: https://phabricator.dev.yugabyte.com/D15196
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
Projects
Row Level Geo Partitioning
Done
Milestone
No milestone
Development

No branches or pull requests
1 participant
@es1024