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

Dawid Weiss commented on LUCENE-2341:
-------------------------------------

One note wrt patch: I would use an explicit pointer over a list of returned 
WordData entries instead of adding them to a local list:

private List<WordData> stemsAcc = new ArrayList<WordData>();

Right now you're shifting the internal array on each call unnecessarily (just 
increase an int ptr instead):

+      termAtt.setEmpty().append(stemsAcc.remove(0).getStem().toString());

getStem() should also be enough since it's a CharSequence, right? No need for 
an intermediate String.

> explore morfologik integration
> ------------------------------
>
>                 Key: LUCENE-2341
>                 URL: https://issues.apache.org/jira/browse/LUCENE-2341
>             Project: Lucene - Java
>          Issue Type: New Feature
>          Components: modules/analysis
>            Reporter: Robert Muir
>            Assignee: Dawid Weiss
>         Attachments: LUCENE-2341.diff, morfologik-stemming-1.5.0.jar
>
>
> Dawid Weiss mentioned on LUCENE-2298 that there is another Polish stemmer 
> available:
> http://sourceforge.net/projects/morfologik/
> This works differently than LUCENE-2298, and ideally would be another option 
> for users.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

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

Reply via email to