You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by no...@apache.org on 2013/06/13 20:42:55 UTC

svn commit: r1492808 - in /lucene/dev/trunk/solr: CHANGES.txt core/src/java/org/apache/solr/cloud/OverseerCollectionProcessor.java

Author: noble
Date: Thu Jun 13 18:42:54 2013
New Revision: 1492808

URL: http://svn.apache.org/r1492808
Log:
SOLR-4925 Collection create throws NPE when 'numShards' param is missing

Modified:
    lucene/dev/trunk/solr/CHANGES.txt
    lucene/dev/trunk/solr/core/src/java/org/apache/solr/cloud/OverseerCollectionProcessor.java

Modified: lucene/dev/trunk/solr/CHANGES.txt
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/CHANGES.txt?rev=1492808&r1=1492807&r2=1492808&view=diff
==============================================================================
--- lucene/dev/trunk/solr/CHANGES.txt (original)
+++ lucene/dev/trunk/solr/CHANGES.txt Thu Jun 13 18:42:54 2013
@@ -139,6 +139,8 @@ Bug Fixes
 * SOLR-4915: The root cause should be returned to the user when a SolrCore create call fails.
   (Mark Miller)
 
+* SOLR-4925 : Collection create throws NPE when 'numShards' param is missing (Noble Paul)
+
 Other Changes
 ----------------------
 

Modified: lucene/dev/trunk/solr/core/src/java/org/apache/solr/cloud/OverseerCollectionProcessor.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/core/src/java/org/apache/solr/cloud/OverseerCollectionProcessor.java?rev=1492808&r1=1492807&r2=1492808&view=diff
==============================================================================
--- lucene/dev/trunk/solr/core/src/java/org/apache/solr/cloud/OverseerCollectionProcessor.java (original)
+++ lucene/dev/trunk/solr/core/src/java/org/apache/solr/cloud/OverseerCollectionProcessor.java Thu Jun 13 18:42:54 2013
@@ -779,7 +779,7 @@ public class OverseerCollectionProcessor
       throws Exception {
     String str = message.getStr(key);
     try {
-      return str == null ? def : Integer.parseInt(str);
+      return str == null ? def : Integer.valueOf(str);
     } catch (Exception ex) {
       SolrException.log(log, "Could not parse " + key, ex);
       throw ex;