[
https://issues.apache.org/jira/browse/LUCENE-6774?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14727515#comment-14727515
]
Uwe Schindler commented on LUCENE-6774:
---------------------------------------
Setting a context class loader inside "library" code is a big problem and
should never ever done. It is good that Robert found this problem. The code has
to be removed, sorry.
Please let us fix the code by:
# first adding a test
# ask [~dweiss] if there were changes he did after this issue in Morphlogik's
resource loading
If both of this does not work would suggest to:
# let Solr set the context ClassLoader: SOLR-3716
# just load the resource files ourselves and pass them to Dictionary#load().
This is mainly because I am not happy that Morfologik uses context ClassLoader
at all. So I would like to fix this by loading the file ourselves from Class's
classpath (or in the factory using ResourceLoader as provided by Lucene/Solr).
> Remove solr hack in MorfologikFilter
> ------------------------------------
>
> Key: LUCENE-6774
> URL: https://issues.apache.org/jira/browse/LUCENE-6774
> Project: Lucene - Core
> Issue Type: Bug
> Reporter: Robert Muir
> Fix For: 5.3.1
>
> Attachments: LUCENE-6774.patch
>
>
> If solr wants to set the contextClassLoader because its classloading is
> fucked up, then it needs to do this hack itself: it should not be in lucene
> code.
> The current mess prevents use of this analyzer in other environments
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
