[ https://issues.apache.org/jira/browse/LUCENE-10335?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17531873#comment-17531873 ]
Uwe Schindler commented on LUCENE-10335: ---------------------------------------- What I wanted to say: if it breaks in classpath mode, then there's a bug in the refactoring in the issue mentioned above. We focused only on the primarily "public" (documented) APIs. The code was always broken as it constructed filenames and their paths very strange from class names. If it uses default files if customizations are not found, it is also a longstanding bug. This is not how lucebe code should work: always fail hard. > IOUtils.getDecodingReader(Class<?>, String) is broken with modules > ------------------------------------------------------------------ > > Key: LUCENE-10335 > URL: https://issues.apache.org/jira/browse/LUCENE-10335 > Project: Lucene - Core > Issue Type: Bug > Reporter: Dawid Weiss > Assignee: Uwe Schindler > Priority: Major > Fix For: 9.1, 10.0 (main) > > Attachments: LUCENE-10335-1.patch, LUCENE-10335.patch, Screenshot > from 2021-12-25 18-04-55.png > > Time Spent: 18h 40m > Remaining Estimate: 0h > > 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. -- This message was sent by Atlassian Jira (v8.20.7#820007) --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org