rmuir commented on code in PR #12311:
URL: https://github.com/apache/lucene/pull/12311#discussion_r1201270494


##########
gradle/testing/defaults-tests.gradle:
##########
@@ -119,11 +119,16 @@ allprojects {
       if (rootProject.runtimeJavaVersion < JavaVersion.VERSION_16) {
         jvmArgs '--illegal-access=deny'
       }
-      
+
       // Lucene needs to optional modules at runtime, which we want to enforce 
for testing
       // (if the runner JVM does not support them, it will fail tests):
       jvmArgs '--add-modules', 'jdk.unsupported,jdk.management'
 
+      // Enable the vector incubator module on supported Java versions:
+      if 
(rootProject.vectorIncubatorJavaVersions.contains(rootProject.runtimeJavaVersion))
 {
+        jvmArgs '--add-modules', 'jdk.incubator.vector'
+      }
+

Review Comment:
   by the way, the slowness was there from the start, just initially sporadic 
because we only implemented 1 out of 6 functions.
   
   it becomes more noticeable and reproducible now that more functions are 
implemented. for example all 3 byte[] similarities are implemented so if you 
just run `./gradle check` it will never complete because 
`TestHnswByteVectorGraph` will always take an eternity.



-- 
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

Reply via email to