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

David Smiley commented on SOLR-3424:
------------------------------------

Looks good Uwe.
One minor nit is that the URL in the class Javadoc to Commons-Codec's
Language package should be this URL:
http://commons.apache.org/codec/apidocs/org/apache/commons/codec/language/package-summary.html
  which is the one to 1.6; the existing link is older with fewer classes. 
We've got this link in both the FilterFactory & Filter.

Feel free to commit if you want or just leave it to me.
                
> PhoneticFilterFactory threadsafety bug
> --------------------------------------
>
>                 Key: SOLR-3424
>                 URL: https://issues.apache.org/jira/browse/SOLR-3424
>             Project: Solr
>          Issue Type: Bug
>          Components: Schema and Analysis
>    Affects Versions: 3.6, 4.0
>            Reporter: David Smiley
>            Assignee: David Smiley
>            Priority: Minor
>             Fix For: 4.0
>
>         Attachments:
SOLR-3424_PhoneticFilterFactory_threadsafety_bug.patch,
SOLR-3424_PhoneticFilterFactory_threadsafety_bug.patch,
SOLR-3424_PhoneticFilterFactory_threadsafety_bug.patch,
SOLR-3424_PhoneticFilterFactory_threadsafety_bug.patch,
SOLR-3424_PhoneticFilterFactory_threadsafety_bug.patch,
SOLR-3424_PhoneticFilterFactory_threadsafety_bug.patch
>
>
> PhoneticFilterFactory has a static HashMap registry mapping an encoder
name to an implementation. There is a ReentrantLock used when the map is
modified (when the encoder config specifies a class name).  However, this
map, which can be accessed by multiple indexing threads, isn't guarded on
any of the reads, which isn't just the common path but also the error
messages which dump the registry into the error message.
> I realize the likelihood of a problem is extremely slim, but a bug's a
bug.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA
administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira