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

sql: fix error handling for AOST clause #77025

Merged
merged 1 commit into from
Feb 25, 2022

Conversation

rafiss
Copy link
Collaborator

@rafiss rafiss commented Feb 25, 2022

fixes #76979
fixes #76978
fixes #76977
fixes #76975
fixes #76787

The AOST clause logic would previously error for any bounded staleness
query that was used in a prepared statement. This is because prepared
statements use an implicit txn, so the same AOST clause has to be
checked multiple times in the txn. A recent commit refactored
the error handling and caused this bug.

Release note: None

@cockroach-teamcity
Copy link
Member

This change is Reviewable

The AOST clause logix would previously error for any bounded staleness
query that was used in a prepared statement. This is because prepared
statements use an implicit txn, so the same AOST clause has to be
checked multiple times in the txn. A recent commit refactored
the error handling and caused this bug.

Release note: None
@rafiss rafiss force-pushed the fix-bounded-staleness-test branch from fbd9a41 to 7ab7829 Compare February 25, 2022 16:10
@rafiss rafiss requested review from otan and a team February 25, 2022 16:10
@rafiss
Copy link
Collaborator Author

rafiss commented Feb 25, 2022

tftr!

bors r=RichardJCai

@craig
Copy link
Contributor

craig bot commented Feb 25, 2022

Build succeeded:

@craig craig bot merged commit c3ff362 into cockroachdb:master Feb 25, 2022
@rafiss rafiss deleted the fix-bounded-staleness-test branch February 25, 2022 20:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment