Github user arysin commented on a diff in the pull request:

    https://github.com/apache/lucene-solr/pull/187#discussion_r111596732
  
    --- Diff: 
lucene/analysis/morfologik/src/java/org/apache/lucene/analysis/uk/UkrainianMorfologikAnalyzer.java
 ---
    @@ -145,7 +152,7 @@ protected TokenStreamComponents createComponents(String 
fieldName) {
     
       private static Dictionary getDictionary() {
         try {
    -      return 
Dictionary.read(UkrainianMorfologikAnalyzer.class.getResource("ukrainian.dict"));
    +      return 
Dictionary.read(UkrainianMorfologikAnalyzer.class.getResource("/ua/net/nlp/ukrainian.dict"));
    --- End diff --
    
    I am a bit confused on why for class loader getResource() we need relative 
path (surprisingly the full one starting with "/" does not work for me), but 
otherwise it makes sense so I'll push the fix shortly.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to