Larger offset when calculating the next histogram bucket to avoid col… (#139675)

gmarouli · web-flow · commit 7ac049f1985b · 2025-12-18T10:40:14.000+01:00
We generate legacy histograms in `DownsampleIT` and we used to small of a delta when generating the histogram values which caused them to collide. We ensure that the delta between the buckets is more than 1.0 fix that. Fixes #139382
diff --git a/muted-tests.yml b/muted-tests.yml
@@ -391,9 +391,6 @@ tests:
 - class: org.elasticsearch.xpack.inference.integration.AuthorizationTaskExecutorIT
   method: testCreatesEisChatCompletionEndpoint
   issue: https://github.com/elastic/elasticsearch/issues/138764
-- class: org.elasticsearch.xpack.downsample.DownsampleIT
-  method: testAggregateMethod
-  issue: https://github.com/elastic/elasticsearch/issues/139382
 - class: org.elasticsearch.xpack.inference.integration.AuthorizationTaskExecutorIT
   method: testCreatesChatCompletion_AndThenCreatesTextEmbedding
   issue: https://github.com/elastic/elasticsearch/issues/138012
diff --git a/x-pack/plugin/downsample/src/internalClusterTest/java/org/elasticsearch/xpack/downsample/DownsampleIT.java b/x-pack/plugin/downsample/src/internalClusterTest/java/org/elasticsearch/xpack/downsample/DownsampleIT.java
@@ -646,7 +646,7 @@ private static double[] randomHistogramValues(int size) {
         final double[] array = new double[size];
         double minHistogramValue = randomDoubleBetween(0.0, 0.1, true);
         for (int i = 0; i < array.length; i++) {
-            array[i] = minHistogramValue += randomDoubleBetween(0.0, 0.5, true);
+            array[i] = minHistogramValue += randomDoubleBetween(0.1, 10.0, true);
         }
         return array;
     }

Original file line number	Diff line number	Diff line change
`@@ -646,7 +646,7 @@ private static double[] randomHistogramValues(int size) {`
`646`	`646`	`final double[] array = new double[size];`
`647`	`647`	`double minHistogramValue = randomDoubleBetween(0.0, 0.1, true);`
`648`	`648`	`for (int i = 0; i < array.length; i++) {`
`649`		`- array[i] = minHistogramValue += randomDoubleBetween(0.0, 0.5, true);`
	`649`	`+ array[i] = minHistogramValue += randomDoubleBetween(0.1, 10.0, true);`
`650`	`650`	`}`
`651`	`651`	`return array;`
`652`	`652`	`}`