[
https://issues.apache.org/jira/browse/LUCENE-7785?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15969224#comment-15969224
]
ASF GitHub Bot commented on LUCENE-7785:
----------------------------------------
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.
> Move dictionary for Ukrainian analyzer to external dependency
> -------------------------------------------------------------
>
> Key: LUCENE-7785
> URL: https://issues.apache.org/jira/browse/LUCENE-7785
> Project: Lucene - Core
> Issue Type: Improvement
> Reporter: Andriy Rysin
> Assignee: Dawid Weiss
>
> Currently the dictionary for Ukrainian analyzer is a blob in the source tree.
> We should move it out to external dependency, this allows:
> * to have less binaries in the source
> * easier to update the dictionary and track updates
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
