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

Taking a whole-table explicit lock might try to to scan whole table #6055

Open
mbautin opened this issue Oct 15, 2020 · 0 comments
Open

Taking a whole-table explicit lock might try to to scan whole table #6055

mbautin opened this issue Oct 15, 2020 · 0 comments
Assignees
Labels
area/docdb YugabyteDB core features kind/enhancement This is an enhancement of an existing feature priority/medium Medium priority issue

Comments

@mbautin
Copy link
Contributor

mbautin commented Oct 15, 2020

Jira Link: DB-2010
This applies to snapshot isolation. We have code in conflict_resolution.cc that tries to detect conflicts with committed transactions. If the intent we are trying to write is a lock on the whole table, we will currently try to scan all rows of the table in regular RocksDB in order to see if anything has been written after the transaction started. This is really inefficient and we should just look at hybrid time frontier values instead of that (but we'll need consensus frontiers containing committed hybrid times).

Cc @jaki @spolitov

@mbautin mbautin self-assigned this Oct 15, 2020
@bmatican bmatican added area/docdb YugabyteDB core features area/ysql Yugabyte SQL (YSQL) labels Mar 3, 2022
@bmatican bmatican added this to YQL-beta Mar 3, 2022
@yugabyte-ci yugabyte-ci added kind/bug This issue is a bug priority/medium Medium priority issue labels Jun 9, 2022
@yugabyte-ci yugabyte-ci added kind/enhancement This is an enhancement of an existing feature and removed kind/bug This issue is a bug labels Jul 28, 2022
@yugabyte-ci yugabyte-ci removed the area/ysql Yugabyte SQL (YSQL) label Oct 12, 2022
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/enhancement This is an enhancement of an existing feature priority/medium Medium priority issue
Projects
Status: No status
Development

No branches or pull requests

4 participants