-
Notifications
You must be signed in to change notification settings - Fork 4.1k
sql: simple SFU queries slower under read committed than serializable #114566
Copy link
Copy link
Closed
Labels
A-read-committedRelated to the introduction of Read CommittedRelated to the introduction of Read CommittedC-performancePerf of queries or internals. Solution not expected to change functional behavior.Perf of queries or internals. Solution not expected to change functional behavior.GA-blockerT-sql-queriesSQL Queries TeamSQL Queries Teambranch-release-23.2Used to mark GA and release blockers, technical advisories, and bugs for 23.2Used to mark GA and release blockers, technical advisories, and bugs for 23.2v23.2.1
Metadata
Metadata
Assignees
Labels
A-read-committedRelated to the introduction of Read CommittedRelated to the introduction of Read CommittedC-performancePerf of queries or internals. Solution not expected to change functional behavior.Perf of queries or internals. Solution not expected to change functional behavior.GA-blockerT-sql-queriesSQL Queries TeamSQL Queries Teambranch-release-23.2Used to mark GA and release blockers, technical advisories, and bugs for 23.2Used to mark GA and release blockers, technical advisories, and bugs for 23.2v23.2.1
Type
Projects
Status
Done
The new implementation of SELECT FOR UPDATE adds a locking lookup join on top of the SELECT plan. For very simple SFU statements this locking lookup join can add measurable overhead over the old implementation of SFU, which pushed locking down to the initial row fetch.
We should try to push locking down and eliminate the lookup join for at least the simplest queries, where the difference will stand out more.
Jira issue: CRDB-33568