rmuir commented on a change in pull request #82:
URL: https://github.com/apache/lucene/pull/82#discussion_r612869465
##########
File path: gradle/generation/icu.gradle
##########
@@ -215,6 +217,7 @@ configure(project(":lucene:analysis:icu")) {
project.quietExec {
executable "make"
workingDir icuSrcDir
+ // TODO: use tests.jvms or similar instead of hardcoding at 4
Review comment:
thanks i'll try the propertyOrDefault. I'd prefer not to use jvm's
ActiveProcessorCount as i set it to 1 on purpose (because the build uses N
jvms, each one should only try to use the resource of one core, so its not
N^2). This way each of my N jvms only uses minimal compiler and gc threads.
--
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org
