-
Notifications
You must be signed in to change notification settings - Fork 9.2k
HDFS-15731. Reduce threadCount for unit tests to reduce the memory usage #2556
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
Conversation
|
🎊 +1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
|
This looks surprisingly successful. |
ayushtkn
left a comment
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.
Thanx @aajisaka for the changes.
The results do seems quite promising, We should run a couple of times and observe too.
We might need to increase the Jenkins timeouts.
In general, would be good to know what was the previous time. I think only RBF build was ~1 hr, if with HDFS, it used to ~3.5 hrs, Not sure though, Can try increasing to 4 again here itself and retry.
|
💔 -1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
|
If I am catching it correct the difference is about 2-2.5 hours, the max I see is 470 and with 4 threads it is around 338. |
|
Thank you @ayushtkn. I added the elapsed time to the description. |
|
Thanx @aajisaka |
|
Updated the PR to reduce the thread count to 2. Thank you @ayushtkn |
|
🎊 +1 overall
This message was automatically generated. |
ayushtkn
left a comment
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.
Thanx @aajisaka
Changes LGTM.
Do revert the blank line changes before committing. :-)
This reverts commit a699cf9.
|
Thank you @ayushtkn for your review. Merged. |
|
💔 -1 overall
This message was automatically generated. |
…age (#2556) Reviewed-by: Ayush Saxena <[email protected]> (cherry picked from commit 9cf408d)
…age (#2556) Reviewed-by: Ayush Saxena <[email protected]> (cherry picked from commit 9cf408d)
…age (#2556) Reviewed-by: Ayush Saxena <[email protected]> (cherry picked from commit 9cf408d)
JIRA: https://issues.apache.org/jira/browse/HDFS-15731
Reduce the thread count from 4 to 2.