mocobeta commented on pull request #638: URL: https://github.com/apache/lucene/pull/638#issuecomment-1028790164
Thanks for your comment. Minor correction - we currently don't even have DictionaryLoader but the resource loading methods are embedded in two BinaryDictionarys themselves. My intention here is, first simplify the resource loading stuff then break it down into proper methods. I know the current switching between classpath/filepath is not good at all and agree with we should remove it at some point. I'll explore if we can remove the delegation around resource loading without changing the API interface (for 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: 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
