Solr Config MergePolicy/MergeScheduler Naming Bug?

2015-06-30 Thread Mike Drob
I was looking through code for unrelated reasons and this line stuck out to me: https://github.com/apache/lucene-solr/blob/trunk/solr/core/src/java/org/apache/solr/update/SolrIndexConfig.java#L180 if(mergeSchedulerInfo != null) m.put(mergeScheduler,mergeSchedulerInfo.toMap()); if(mergePolicyInfo

Re: Solr Config MergePolicy/MergeScheduler Naming Bug?

2015-06-30 Thread Mike Drob
It looks like this toMap method is only used (transitively) by SolrConfigHandler.handleGet to print the config and show it to a client. Probably safe to add the other elements as well, good catch. On Tue, Jun 30, 2015 at 3:49 PM, Christine Poerschke (BLOOMBERG/ LONDON) cpoersc...@bloomberg.net

Re: Solr Config MergePolicy/MergeScheduler Naming Bug?

2015-06-30 Thread Mike Drob
https://issues.apache.org/jira/browse/SOLR-7741 On Tue, Jun 30, 2015 at 3:57 PM, Mike Drob mad...@cloudera.com wrote: It looks like this toMap method is only used (transitively) by SolrConfigHandler.handleGet to print the config and show it to a client. Probably safe to add the other elements