[ 
https://issues.apache.org/jira/browse/LUCENE-6833?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15030824#comment-15030824
 ] 

David Smiley commented on LUCENE-6833:
--------------------------------------

When I try to run the maven build, it fails because essentially morfologik 
doesn't have "test-files" on the classpath.  I had to modify the morfologik pom 
(via our template) to include this, like so:
{code:xml}
<testResource>
        <directory>${module-path}/src/test-files</directory>
 </testResource>
{code}
(which is done similarly for some other modules, e.g. velocity).

I'm confused why I'm discovering this now... so strange. Any ideas what's going 
on?  Pending what we find, I plan to commit this fix.

> Upgrade morfologik to version 2.0.1, simplify MorfologikFilter's dictionary 
> lookup
> ----------------------------------------------------------------------------------
>
>                 Key: LUCENE-6833
>                 URL: https://issues.apache.org/jira/browse/LUCENE-6833
>             Project: Lucene - Core
>          Issue Type: Improvement
>            Reporter: Dawid Weiss
>            Assignee: Dawid Weiss
>            Priority: Minor
>             Fix For: Trunk, 5.4
>
>         Attachments: LUCENE-6833.patch, LUCENE-6833.patch
>
>
> This is a follow-up to Uwe's work on LUCENE-6774. 
> This patch updates the code to use Morfologik stemming version 2.0.1, which 
> removes the "automatic" lookup of classpath-relative dictionary resources in 
> favor of an explicit InputStream or URL. So the user code is explicitly 
> responsible to provide these resources, reacting to missing files, etc.
> There were no other "default" dictionaries in Morfologik other than the 
> Polish dictionary so I also cleaned up the filter code from a number of 
> attributes that were, to me, confusing. 
> * {{MorfologikFilterFactory}} now accepts an (optional) {{dictionary}} 
> attribute which contains an explicit name of the dictionary resource to load. 
> The resource is loaded with a {{ResourceLoader}} passed to the {{inform(..)}} 
> method, so the final location depends on the resource loader.
> * There is no way to load the dictionary and metadata separately (this isn't 
> at all useful).
> * If the {{dictionary}} attribute is missing, the filter loads the Polish 
> dictionary by default (since most people would be using Morfologik for 
> stemming Polish anyway).
> This patch is *not* backward compatible, but it attempts to provide useful 
> feedback on initialization: if the removed attributes were used, it points at 
> this JIRA issue, so it should be clear what to change and how.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

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

Reply via email to