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

[dist-txns] optimize selection of txn status tablet such that it's local to current txn's initiator #331

Closed
spolitov opened this issue Jun 15, 2018 · 0 comments
Closed

[dist-txns] optimize selection of txn status tablet such that it's local to current txn's initiator #331

spolitov opened this issue Jun 15, 2018 · 0 comments
Assignees
@spolitov
Labels
kind/enhancement This is an enhancement of an existing feature

Comments

@spolitov
Copy link
Contributor

Currently, when a distributed transaction is started in ybclient in CQL proxy, ybclient makes a RPC call (yb.master.MasterService.GetTableLocations) to master to fetch the IDs of all tablets of the "system.transactions" table and pick a random tablet to use as the status tablet for the transaction.

For performance, we should pick a tablet that is local to CQL proxy if possible so the begin transaction and commit will be local.

Also, we should cache the result to avoid the RPC call to master.
@kmuthukk kmuthukk added the kind/enhancement This is an enhancement of an existing feature label Jun 15, 2018
yugabyte-ci pushed a commit that referenced this issue Jun 18, 2018
@spolitov
(#331) ENG-3407: Pick local transaction status tablet
28442cc 
Summary: Prefer tablets that has local leader, when picking status tablet for transaction.

Test Plan: Jenkins

Reviewers: mikhail, pritam.damania, robert

Reviewed By: robert

Subscribers: bogdan, ybase, bharat

Differential Revision: https://phabricator.dev.yugabyte.com/D4972
mbautin pushed a commit that referenced this issue Jul 11, 2019
@spolitov @mbautin
This commit provides additional functionality that is logically relat…
01f7b08 
…ed to the

earlier commit 28442cc

Original commit message:

(#331) ENG-3407: Pick local transaction status tablet

Summary: Prefer tablets that has local leader, when picking status tablet for transaction.

Test Plan: Jenkins

Reviewers: mikhail, pritam.damania, robert

Reviewed By: robert

Subscribers: bogdan, ybase, bharat

Differential Revision: https://phabricator.dev.yugabyte.com/D4972
mbautin pushed a commit to mbautin/yugabyte-db that referenced this issue Jul 16, 2019
@spolitov
(yugabyte#331) ENG-3407: Pick local transaction status tablet
a383a5b 
Summary: Prefer tablets that has local leader, when picking status tablet for transaction.

Test Plan: Jenkins

Reviewers: mikhail, pritam.damania, robert

Reviewed By: robert

Subscribers: bogdan, ybase, bharat

Differential Revision: https://phabricator.dev.yugabyte.com/D4972

Note:
This commit provides additional functionality that is logically related to
the earlier commit yugabyte@28442cc
and supersedes the commit yugabyte@01f7b08
jasonyb pushed a commit that referenced this issue Jun 11, 2024
@Naeem-Akhter
Merge pull request #331 from ibrarahmad/PGSM-518
2838eaa 
PG-518: Internal Functions should NOT be visible in PGSM API.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@spolitov spolitov

Labels
kind/enhancement This is an enhancement of an existing feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@spolitov @kmuthukk