Nits

Author: jtuglu1

benchmarks/src/test/java/org/apache/druid/benchmark/SpectatorHistogramAggregatorBenchmark.java

@@ -321,7 +321,7 @@ public static void main(String[] args) throws RunnerException
         .include(SpectatorHistogramAggregatorBenchmark.class.getSimpleName())
         .forks(1)
         .warmupIterations(1)
-        .measurementIterations(2)
+        .measurementIterations(10)
         .build();
     new Runner(opt).run();
   }

extensions-contrib/spectator-histogram/src/main/java/org/apache/druid/spectator/histogram/SpectatorHistogram.java

@@ -278,7 +278,7 @@ byte[] toBytes()
 
   public void insert(Number num)
   {
-    this.add(PercentileBuckets.indexOf(num.longValue()), 1L);
+    this.insert(num.longValue());
   }
 
   public void insert(long num)

extensions-contrib/spectator-histogram/src/main/java/org/apache/druid/spectator/histogram/SpectatorHistogramNumericVectorizedAggregator.java

@@ -74,6 +74,10 @@ public void aggregate(ByteBuffer buffer, int numRows, int[] positions, @Nullable
     boolean hasNulls = isNull != null;
 
     final Int2ObjectMap<SpectatorHistogram> histMap = innerAggregator.get(buffer);
+    if (histMap == null) {
+      return;
+    }
+
     for (int i = 0; i < numRows; ++i) {
       int rowIndex = rows != null ? rows[i] : i;
       boolean rowIsNull = hasNulls && isNull[rowIndex];

extensions-contrib/spectator-histogram/src/main/java/org/apache/druid/spectator/histogram/SpectatorHistogramVectorizedAggregator.java

@@ -75,6 +75,10 @@ public void aggregate(ByteBuffer buffer, int numRows, int[] positions, @Nullable
     }
 
     final Int2ObjectMap<SpectatorHistogram> histMap = innerAggregator.get(buffer);
+    if (histMap == null) {
+      return;
+    }
+
     for (int i = 0; i < numRows; ++i) {
       int rowIndex = rows != null ? rows[i] : i;
       Object other = vector[rowIndex];