[GitHub] [netbeans] mbien commented on pull request #3558: maven indexing lucene renovation

[GitBox <gi...@apache.org>]

mbien commented on pull request #3558:
URL: https://github.com/apache/netbeans/pull/3558#issuecomment-1065082967


   > Maven continues to support JDK8 and even older. E.g. Maven itself must be [using compatible version of Lucene](https://github.com/apache/maven-indexer/blob/master/pom.xml#L95). I suggest NetBeans to use the [same version](https://github.com/apache/maven-indexer/blob/master/pom.xml#L95).
   
   you mean maven-indexer? Thats exactly what this PR does. Latest 8.x is in use which is the same what maven-indexer 6.1x uses, Compatibility windows are described in the PR text and why this all had to be done in one step (5.x -> 8.x). The encountered problems are listed too. 
   
   The only potential showstopper remaining (IMO) is the discovered bug in lucene's hashcode logic, although it got quickly fixed thanks to lucene devs after I filed the issue, it doesn't really help us if it won't be backported to 8.x which we are stuck to due to the JDK 8 requirement of this module.


-- 
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: notifications-unsubscribe@netbeans.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org
For additional commands, e-mail: notifications-help@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists