-
Notifications
You must be signed in to change notification settings - Fork 28.5k
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-24531][TESTS] Replace 2.3.0 version with 2.3.1 #21543
Conversation
cc @vanzin |
Test build #91729 has finished for PR 21543 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.
but this seems fragile
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
Thanks! Merged to master
This should go into branch-2.3 too. I also think this is a little fragile (which is why I filed SPARK-24532), and I've been playing with some alternative code, but it feels like a bit too much code to avoid a little extra work on every release... |
## What changes were proposed in this pull request? The PR updates the 2.3 version tested to the new release 2.3.1. ## How was this patch tested? existing UTs Author: Marco Gaido <marcogaido91@gmail.com> Closes apache#21543 from mgaido91/patch-1.
## What changes were proposed in this pull request? The PR updates the 2.3 version tested to the new release 2.3.1. ## How was this patch tested? existing UTs Author: Marco Gaido <marcogaido91@gmail.com> Closes apache#21543 from mgaido91/patch-1.
What changes were proposed in this pull request?
The PR updates the 2.3 version tested to the new release 2.3.1.
How was this patch tested?
existing UTs