matthiasblaesing commented on PR #9207: URL: https://github.com/apache/netbeans/pull/9207#issuecomment-3941721165
> I've done some testing today. The old `$NETBEANS_CACHE_DIR/index` should be removed if there is one exists before this change, otherwise there are a bunch of `IndexFormatTooOldException` would be thrown. Though that should not be an issue. This is just an FYI for others reading this PR. Yeah. I noticed that when I switch between 10 and 9 for testing. I'll see if I can up with a method to allow a smother up/downgrade (most probably this will mean: nuke the exiting index). > `MemoryIndex` still throws an `AlreadyClosedException`, after a file has been modified and auto-completion has been invoked. Found the issue. Somewhere between multiple iterations I move to a try-with-resource setup and this was a remaining not reverted change. The readers in MemoryIndex and LuceneIndex are shared and must not be closed at the use-sites. Should be fixed now. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected] For further information about the NetBeans mailing lists, visit: https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists
