[ https://issues.apache.org/jira/browse/LUCENE-2051?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12778190#action_12778190 ]
Robert Muir commented on LUCENE-2051: ------------------------------------- bq. This is different. the StopawareAnalyzer#getStopwords() is an instance method to get the "current" stopword set of the instance. while the ones I introduced here are static to get the default set instead. We need to provide a replacement for the public static final Sting[] stuff for deprecation an I thing they have to be there. thoughts? right, but still, will they this static method be supported after refactoring to StopAwareAnalyzer? > Contrib Analyzer Setters should be deprecated and replace with ctor arguments > ----------------------------------------------------------------------------- > > Key: LUCENE-2051 > URL: https://issues.apache.org/jira/browse/LUCENE-2051 > Project: Lucene - Java > Issue Type: Improvement > Components: contrib/analyzers > Affects Versions: 2.9.1 > Reporter: Simon Willnauer > Assignee: Simon Willnauer > Priority: Minor > Fix For: 3.0 > > Attachments: LUCENE-2051.patch, LUCENE-2051.patch > > > Some analyzers in contrib provide setters for stopword / stem exclusion sets > / hashtables etc. Those setters should be deprecated as they yield unexpected > behaviour. The way they work is they set the reusable token stream instance > to null in a thread local cache which only affects the tokenstream in the > current thread. Analyzers itself should be immutable except of the > threadlocal. > will attach a patch soon. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. --------------------------------------------------------------------- To unsubscribe, e-mail: java-dev-unsubscr...@lucene.apache.org For additional commands, e-mail: java-dev-h...@lucene.apache.org