uschindler commented on code in PR #11937:
URL: https://github.com/apache/lucene/pull/11937#discussion_r1023954625
##########
gradle/hacks/turbocharge-jvm-opts.gradle:
##########
@@ -38,4 +39,20 @@ allprojects {
jvmArgs += vmOpts
}
-}
\ No newline at end of file
+
+ // Tweak javac to not be too resource-hungry.
+ // This applies to any JVM when javac runs forked (e.g. error-prone)
+ // Avoiding the fork entirely is best.
+ tasks.withType(JavaCompile) { JavaCompile task ->
+ if (task.path == ":lucene:core:compileMain19Java") {
Review Comment:
> > @rmuir altJvm is possible on 10.x as well, just pass a path to JDK17 via
-Pruntime.java.home=/my/jdk/17 and it'll use that JVM.
>
> yes, i know, but i was trying to test the grid of `altjvm=True/False X
errorprone=True/False` on 10.x, you can't do altjvm=True + errorprone=True, it
gets disabled due to some workaround. So I tested that on 9.x
>
> thanks for looking at it, as long as jenkins is happy we are good.
Yes altjvm does not work with errorprone due to a bug in the errorprone
plugin. the issue has to do with the open modules. Maybe it is the same issue
like with -J and no -J.
We disabled it. For Gradle's Toolkit, theres support in the errorprone
plugin. It just does not work with our altjvm support.
--
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: issues-unsubscr...@lucene.apache.org
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
