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
[SPARK-29357][SQL][TESTS] Fix flaky test by changing to use AtomicLong #26020
Conversation
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.
LGTM
Test build #111765 has finished for PR 26020 at commit
|
retest this please |
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.
@xuanyuanking, can you file a new JIRA for trackability? SPARK-25159 was fixed in 2.4.0 and fixed version of the JIRA gets confused when this is merged.
Sure, fill the new JIRA SPARK-29357 to track this. |
Test build #111769 has finished for PR 26020 at commit
|
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.
+1, LGTM. Merged to master.
Thanks! |
What changes were proposed in this pull request?
Change to use AtomicLong instead of a var in the test.
Why are the changes needed?
Fix flaky test.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Existing UT.