-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Add reusable smoke test coverage for concurrent UPDATE #10368
Add reusable smoke test coverage for concurrent UPDATE #10368
Conversation
verifyConcurrentUpdateFailurePermissible(trinoException); | ||
} | ||
catch (Throwable verifyFailure) { | ||
if (trinoException != e && verifyFailure != e) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why trinoException != e
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
because trinoException == e
implies that attaching e
as suppressed is kind of pointless.
verifyConcurrentUpdateFailurePermissible
should be rethrowing trinoException
553ebf8
to
a6377b8
Compare
285b10a
to
b9f63a7
Compare
public void testUpdateRowConcurrently() | ||
throws Exception | ||
{ | ||
// TODO (https://github.com/trinodb/trino/issues/10518) test this with a TestHiveConnectorTest version that creates ACID tables by default, or in some other way |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
b9f63a7
to
cbac3c5
Compare
No description provided.