-
Notifications
You must be signed in to change notification settings - Fork 28.3k
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-26306][TEST][BUILD] More memory to de-flake SorterSuite #23425
Conversation
Test build #100634 has finished for PR 23425 at commit
|
retest this please. |
Test build #100637 has finished for PR 23425 at commit
|
Thank you for making this PR. I know that we are very conservative in this area, but I'm +1 for this PR. @rxin and @aarondav . Since SPARK-26306 is related to SPARK-5984, could you give us some advice for this PR? |
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 |
## What changes were proposed in this pull request? Increase test memory to avoid OOM in TimSort-related tests. ## How was this patch tested? Existing tests. Closes #23425 from srowen/SPARK-26306. Authored-by: Sean Owen <sean.owen@databricks.com> Signed-off-by: Sean Owen <sean.owen@databricks.com>
It's great. Thank you, @srowen ! |
## What changes were proposed in this pull request? Increase test memory to avoid OOM in TimSort-related tests. ## How was this patch tested? Existing tests. Closes apache#23425 from srowen/SPARK-26306. Authored-by: Sean Owen <sean.owen@databricks.com> Signed-off-by: Sean Owen <sean.owen@databricks.com>
What changes were proposed in this pull request?
Increase test memory to avoid OOM in TimSort-related tests.
How was this patch tested?
Existing tests.