You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by ab...@apache.org on 2018/09/17 10:09:29 UTC

[44/44] lucene-solr:jira/solr-12709: SOLR-12709: Fix NPE.

SOLR-12709: Fix NPE.


Project: http://git-wip-us.apache.org/repos/asf/lucene-solr/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucene-solr/commit/93f67266
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/93f67266
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/93f67266

Branch: refs/heads/jira/solr-12709
Commit: 93f67266fb851daa33e807eddb897e1f66aed0e8
Parents: 5fcd910
Author: Andrzej Bialecki <ab...@apache.org>
Authored: Mon Sep 17 12:08:23 2018 +0200
Committer: Andrzej Bialecki <ab...@apache.org>
Committed: Mon Sep 17 12:08:23 2018 +0200

----------------------------------------------------------------------
 .../autoscaling/sim/SimClusterStateProvider.java    | 16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/93f67266/solr/core/src/test/org/apache/solr/cloud/autoscaling/sim/SimClusterStateProvider.java
----------------------------------------------------------------------
diff --git a/solr/core/src/test/org/apache/solr/cloud/autoscaling/sim/SimClusterStateProvider.java b/solr/core/src/test/org/apache/solr/cloud/autoscaling/sim/SimClusterStateProvider.java
index 9a32599..693838e 100644
--- a/solr/core/src/test/org/apache/solr/cloud/autoscaling/sim/SimClusterStateProvider.java
+++ b/solr/core/src/test/org/apache/solr/cloud/autoscaling/sim/SimClusterStateProvider.java
@@ -1873,8 +1873,12 @@ public class SimClusterStateProvider implements ClusterStateProvider {
               if (ri != null) {
                 Number numDocs = (Number)ri.getVariable("SEARCHER.searcher.numDocs");
                 Number numBytes = (Number)ri.getVariable("INDEX.sizeInBytes");
-                inactiveDocs.addValue(numDocs.doubleValue());
-                inactiveBytes.addValue(numBytes.doubleValue());
+                if (numDocs != null) {
+                  inactiveDocs.addValue(numDocs.doubleValue());
+                }
+                if (numBytes != null) {
+                  inactiveBytes.addValue(numBytes.doubleValue());
+                }
               }
             }
             continue;
@@ -1902,11 +1906,15 @@ public class SimClusterStateProvider implements ClusterStateProvider {
             Number numDocs = (Number)ri.getVariable("SEARCHER.searcher.numDocs");
             Number delDocs = (Number)ri.getVariable("SEARCHER.searcher.deleteDocs");
             Number numBytes = (Number)ri.getVariable("INDEX.sizeInBytes");
-            docs.addValue(numDocs.doubleValue());
+            if (numDocs != null) {
+              docs.addValue(numDocs.doubleValue());
+            }
             if (delDocs != null) {
               deletedDocs += delDocs.longValue();
             }
-            bytes.addValue(numBytes.doubleValue());
+            if (numBytes != null) {
+              bytes.addValue(numBytes.doubleValue());
+            }
           }
         }
         perColl.put("shardsState", shardState);