[GitHub] [drill] cgivre commented on pull request #2301: Improve MAVEN build Performance

[GitBox <gi...@apache.org>]

cgivre commented on pull request #2301:
URL: https://github.com/apache/drill/pull/2301#issuecomment-912796409


   > @ChenZhangg we have already use parallel builds. And this 2 forks right now:
   > https://github.com/apache/drill/blob/master/pom.xml#L52
   > On the other hand, this is the issue, that some test fails intermittently right now for parallel maven builds. I am going to solve it in scope of: #2293
   > 
   > About `-T 1C`, it means to use the same number of forks as the number of cores on machine. But it also means we will use the same number of JVMs and the memory usage will be increased. Since Drill is memory cost tool and not all machines has enough amount of memory this option is not sufficient for all machines. For instance: user can have laptop with two physical cores, but with Intel multi-threading that number will be 4. The default memory usage for Drill in scope of one JVM is up to 4Gb. So for laptops with 8Gb memory the build will fail.
   
   @vdiravka Is that a -1 for this PR?  Should we close?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@drill.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org