mocobeta commented on pull request #643: URL: https://github.com/apache/lucene/pull/643#issuecomment-1030612811
I added `@SuppressWarnings("removal")` to tests that invoke deprecated methods for now - to keep the tests as-is on branch_9x. I will remove the obsolete methods and change tests in a separate patch (after backporting this). Also, the suggested `ResourceSupplier<T>` interface works very well, thanks. As for the wrapping checked exception trick (type erasure plays the main role there, maybe?), it's cool but seems too much for me. -- 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