uschindler commented on code in PR #12765:
URL: https://github.com/apache/lucene/pull/12765#discussion_r1382621462


##########
lucene/core/src/java/org/apache/lucene/internal/vectorization/VectorizationProvider.java:
##########
@@ -111,6 +111,12 @@ static VectorizationProvider lookup(boolean testMode) {
                 + Locale.getDefault());
         return new DefaultVectorizationProvider();
       }
+      // only use vector module with Hotspot VM

Review Comment:
   the order is a bit different than in @rmuir's PR #12764: Here it exits early 
and does not even suggests to add the incubator command line flags. Therefore 
the check for a readable vector module is done after the Hotspot check. 



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