uschindler commented on a change in pull request #18:
URL: https://github.com/apache/lucene/pull/18#discussion_r594205187



##########
File path: gradle/defaults-java.gradle
##########
@@ -60,3 +60,14 @@ allprojects {
     }
   }
 }
+
+configure(project(":lucene:core")) {
+  plugins.withType(JavaPlugin) {

Review comment:
       I feel the same for MMapDirectory v2: "it should really go into Lucene 
Core", with some trick, but at the moment I have to wait. Netty has something 
similar at moment, you have to download a separate JAR and add it to your 
classpath.
   
   So one way to implement this: Add a separate lucene module that's optional 
and use some "plug in" mechanism (e.g., SPI). People who want to use it must 
use the exact correct version for their JDK version and must add some command 
line flag to use it (enable it in their JDK).




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

Reply via email to