[LUCENE-10335] IOUtils.getDecodingReader(Class<?>, String) is broken with modules

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 9.1, 10.0 (main)
• Lucene Fields: New
• Assignee: Uwe Schindler
• Reporter: Dawid Weiss
• Created: 22/Dec/21 20:20
• Updated: 28/Aug/22 16:33
• Resolved: 03/Jan/22 10:51

Description

This method calls clazz.getResourceAsStream() but in a modular application the method won't see any of the resources in clazz's module, causing an NPE. We should deprecate or even remove this method entirely, leaving only getDecodingReader(InputStream) and opening the resource on the caller's side.

Issue Links

• relates to: LUCENE-10328 Module path for compiling and running tests is wrong
• Testing discovered: LUCENE-10348 Make stopwords resources from analyzers modules visible to ClasspathResourceLoader and ModuleResourceLoader
• links to: GitHub Pull Request #567