msokolov commented on PR #868: URL: https://github.com/apache/lucene/pull/868#issuecomment-1118443078
> In case you ask: This works both with classpath and module usage. The caller-sensitive parts are `Class#getResource(String)`, `ClassLoader#getResource(String)`, and `Module#getResource(String)`. The returned URL is free to use anywhere so it separates concerns like the `IOSupplier` or `Path`. I was going to ask about this :) Didn't know about Module#getResource ... Anyway this approach seems sound. Although it's a little less general than the Stream-based approach, it does handle all the known use cases cleanly. -- 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
