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

Ishan Chattopadhyaya commented on SOLR-7688:
--------------------------------------------

bq. My patch changes it to a LinkedHashMap, adds a test, and also fixes up some 
params.
+1

bq. The double initialization issue looks interesting, thanks for pointing that 
out. Assuming that is the case, though, the debugFilter is also double 
initialized and should be handled in a similar way? Seems like that should be 
handled in a separate issue.

Sure, that makes sense. I'll try to reproduce again and open a new issue for 
that.

> JettyConfig cannot be used with extra filters
> ---------------------------------------------
>
>                 Key: SOLR-7688
>                 URL: https://issues.apache.org/jira/browse/SOLR-7688
>             Project: Solr
>          Issue Type: Improvement
>    Affects Versions: 5.1
>            Reporter: Gregory Chanan
>            Assignee: Gregory Chanan
>         Attachments: SOLR-7688.patch, SOLR-7688.patch
>
>
> Before SOLR-7166, users could create a MiniSolrCloudCluster with extra 
> filters by specifying their own SortedMap and comparator (since Class<? 
> extends Filter> is not comparable).  JettyConfigs allow you to specify filter 
> classes to add, but they don't work because JettyConfigs manages its own 
> filter map using TreeMap.  Thus, there is no way to specify a comparator and 
> it fails at runtime with "Class cannot be cast to java.lang.comparable."



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

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to