You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@datasketches.apache.org by le...@apache.org on 2022/03/26 02:26:09 UTC
[datasketches-java] 07/12: Interim 7 Fixes a bug introduced in Interim 6
This is an automated email from the ASF dual-hosted git repository.
leerho pushed a commit to branch DirectKllDouble
in repository https://gitbox.apache.org/repos/asf/datasketches-java.git
commit d51ee56700d82b69f7ddda80c97e68b14f34b0ce
Author: Lee Rhodes <le...@users.noreply.github.com>
AuthorDate: Sat Mar 5 13:23:04 2022 -0800
Interim 7 Fixes a bug introduced in Interim 6
---
.../apache/datasketches/kll/KllDoublesSketch.java | 49 +++++++++++-----------
.../apache/datasketches/kll/KllFloatsSketch.java | 44 +++++++++----------
.../kll/{HeapKllSketch.java => KllHeapSketch.java} | 4 +-
.../org/apache/datasketches/kll/KllHelper.java | 4 +-
.../{PreambleUtil.java => KllPreambleUtil.java} | 4 +-
.../org/apache/datasketches/kll/KllSketch.java | 14 +++----
.../datasketches/kll/KllDoublesSketchTest.java | 24 +++++------
.../datasketches/kll/KllFloatsSketchTest.java | 22 +++++-----
.../org/apache/datasketches/kll/KllHelperTest.java | 2 +-
.../apache/datasketches/kll/MiscDoublesTest.java | 2 +-
.../apache/datasketches/kll/MiscFloatsTest.java | 2 +-
11 files changed, 85 insertions(+), 86 deletions(-)
diff --git a/src/main/java/org/apache/datasketches/kll/KllDoublesSketch.java b/src/main/java/org/apache/datasketches/kll/KllDoublesSketch.java
index 76ae938..ef87d96 100644
--- a/src/main/java/org/apache/datasketches/kll/KllDoublesSketch.java
+++ b/src/main/java/org/apache/datasketches/kll/KllDoublesSketch.java
@@ -22,34 +22,34 @@ package org.apache.datasketches.kll;
import static java.lang.Math.max;
import static java.lang.Math.min;
import static org.apache.datasketches.Util.isOdd;
-import static org.apache.datasketches.kll.PreambleUtil.DATA_START_ADR_DOUBLE;
-import static org.apache.datasketches.kll.PreambleUtil.DATA_START_ADR_SINGLE_ITEM;
-import static org.apache.datasketches.kll.PreambleUtil.DEFAULT_K;
-import static org.apache.datasketches.kll.PreambleUtil.DOUBLES_SKETCH_BIT_MASK;
-import static org.apache.datasketches.kll.PreambleUtil.DY_MIN_K_SHORT_ADR;
-import static org.apache.datasketches.kll.PreambleUtil.EMPTY_BIT_MASK;
-import static org.apache.datasketches.kll.PreambleUtil.FAMILY_BYTE_ADR;
-import static org.apache.datasketches.kll.PreambleUtil.FLAGS_BYTE_ADR;
-import static org.apache.datasketches.kll.PreambleUtil.K_SHORT_ADR;
-import static org.apache.datasketches.kll.PreambleUtil.LEVEL_ZERO_SORTED_BIT_MASK;
-import static org.apache.datasketches.kll.PreambleUtil.M_BYTE_ADR;
-import static org.apache.datasketches.kll.PreambleUtil.NUM_LEVELS_BYTE_ADR;
-import static org.apache.datasketches.kll.PreambleUtil.N_LONG_ADR;
-import static org.apache.datasketches.kll.PreambleUtil.PREAMBLE_INTS_BYTE_ADR;
-import static org.apache.datasketches.kll.PreambleUtil.PREAMBLE_INTS_DOUBLE;
-import static org.apache.datasketches.kll.PreambleUtil.PREAMBLE_INTS_EMPTY_SINGLE;
-import static org.apache.datasketches.kll.PreambleUtil.SERIAL_VERSION_EMPTY_FULL;
-import static org.apache.datasketches.kll.PreambleUtil.SERIAL_VERSION_SINGLE;
-import static org.apache.datasketches.kll.PreambleUtil.SER_VER_BYTE_ADR;
-import static org.apache.datasketches.kll.PreambleUtil.SINGLE_ITEM_BIT_MASK;
-import static org.apache.datasketches.kll.PreambleUtil.UPDATABLE_BIT_MASK;
+import static org.apache.datasketches.kll.KllPreambleUtil.DATA_START_ADR_DOUBLE;
+import static org.apache.datasketches.kll.KllPreambleUtil.DATA_START_ADR_SINGLE_ITEM;
+import static org.apache.datasketches.kll.KllPreambleUtil.DEFAULT_K;
+import static org.apache.datasketches.kll.KllPreambleUtil.DOUBLES_SKETCH_BIT_MASK;
+import static org.apache.datasketches.kll.KllPreambleUtil.DY_MIN_K_SHORT_ADR;
+import static org.apache.datasketches.kll.KllPreambleUtil.EMPTY_BIT_MASK;
+import static org.apache.datasketches.kll.KllPreambleUtil.FAMILY_BYTE_ADR;
+import static org.apache.datasketches.kll.KllPreambleUtil.FLAGS_BYTE_ADR;
+import static org.apache.datasketches.kll.KllPreambleUtil.K_SHORT_ADR;
+import static org.apache.datasketches.kll.KllPreambleUtil.LEVEL_ZERO_SORTED_BIT_MASK;
+import static org.apache.datasketches.kll.KllPreambleUtil.M_BYTE_ADR;
+import static org.apache.datasketches.kll.KllPreambleUtil.NUM_LEVELS_BYTE_ADR;
+import static org.apache.datasketches.kll.KllPreambleUtil.N_LONG_ADR;
+import static org.apache.datasketches.kll.KllPreambleUtil.PREAMBLE_INTS_BYTE_ADR;
+import static org.apache.datasketches.kll.KllPreambleUtil.PREAMBLE_INTS_DOUBLE;
+import static org.apache.datasketches.kll.KllPreambleUtil.PREAMBLE_INTS_EMPTY_SINGLE;
+import static org.apache.datasketches.kll.KllPreambleUtil.SERIAL_VERSION_EMPTY_FULL;
+import static org.apache.datasketches.kll.KllPreambleUtil.SERIAL_VERSION_SINGLE;
+import static org.apache.datasketches.kll.KllPreambleUtil.SER_VER_BYTE_ADR;
+import static org.apache.datasketches.kll.KllPreambleUtil.SINGLE_ITEM_BIT_MASK;
+import static org.apache.datasketches.kll.KllPreambleUtil.UPDATABLE_BIT_MASK;
import java.util.Arrays;
import org.apache.datasketches.Family;
import org.apache.datasketches.SketchesArgumentException;
import org.apache.datasketches.Util;
-import org.apache.datasketches.kll.PreambleUtil.MemoryCheck;
+import org.apache.datasketches.kll.KllPreambleUtil.MemoryCheck;
import org.apache.datasketches.memory.Memory;
import org.apache.datasketches.memory.WritableMemory;
@@ -57,7 +57,7 @@ import org.apache.datasketches.memory.WritableMemory;
* Please refer to the documentation in the package-info:<br>
* {@link org.apache.datasketches.kll}
*/
-public class KllDoublesSketch extends HeapKllSketch {
+public class KllDoublesSketch extends KllHeapSketch {
// Specific to the doubles sketch
private double[] items_; // the continuous array of double items
@@ -764,8 +764,7 @@ public class KllDoublesSketch extends HeapKllSketch {
final int theShift = freeSpaceAtBottom - outlevels[0];
if (getLevelsArray().length < finalNumLevels + 1) {
-
-;
+ setLevelsArray(new int[finalNumLevels + 1]);
}
for (int lvl = 0; lvl < finalNumLevels + 1; lvl++) { // includes the "extra" index
diff --git a/src/main/java/org/apache/datasketches/kll/KllFloatsSketch.java b/src/main/java/org/apache/datasketches/kll/KllFloatsSketch.java
index e550c29..bb71c83 100644
--- a/src/main/java/org/apache/datasketches/kll/KllFloatsSketch.java
+++ b/src/main/java/org/apache/datasketches/kll/KllFloatsSketch.java
@@ -22,33 +22,33 @@ package org.apache.datasketches.kll;
import static java.lang.Math.max;
import static java.lang.Math.min;
import static org.apache.datasketches.Util.isOdd;
-import static org.apache.datasketches.kll.PreambleUtil.DATA_START_ADR_FLOAT;
-import static org.apache.datasketches.kll.PreambleUtil.DATA_START_ADR_SINGLE_ITEM;
-import static org.apache.datasketches.kll.PreambleUtil.DEFAULT_K;
-import static org.apache.datasketches.kll.PreambleUtil.DY_MIN_K_SHORT_ADR;
-import static org.apache.datasketches.kll.PreambleUtil.EMPTY_BIT_MASK;
-import static org.apache.datasketches.kll.PreambleUtil.FAMILY_BYTE_ADR;
-import static org.apache.datasketches.kll.PreambleUtil.FLAGS_BYTE_ADR;
-import static org.apache.datasketches.kll.PreambleUtil.K_SHORT_ADR;
-import static org.apache.datasketches.kll.PreambleUtil.LEVEL_ZERO_SORTED_BIT_MASK;
-import static org.apache.datasketches.kll.PreambleUtil.M_BYTE_ADR;
-import static org.apache.datasketches.kll.PreambleUtil.NUM_LEVELS_BYTE_ADR;
-import static org.apache.datasketches.kll.PreambleUtil.N_LONG_ADR;
-import static org.apache.datasketches.kll.PreambleUtil.PREAMBLE_INTS_BYTE_ADR;
-import static org.apache.datasketches.kll.PreambleUtil.PREAMBLE_INTS_EMPTY_SINGLE;
-import static org.apache.datasketches.kll.PreambleUtil.PREAMBLE_INTS_FLOAT;
-import static org.apache.datasketches.kll.PreambleUtil.SERIAL_VERSION_EMPTY_FULL;
-import static org.apache.datasketches.kll.PreambleUtil.SERIAL_VERSION_SINGLE;
-import static org.apache.datasketches.kll.PreambleUtil.SER_VER_BYTE_ADR;
-import static org.apache.datasketches.kll.PreambleUtil.SINGLE_ITEM_BIT_MASK;
-import static org.apache.datasketches.kll.PreambleUtil.UPDATABLE_BIT_MASK;
+import static org.apache.datasketches.kll.KllPreambleUtil.DATA_START_ADR_FLOAT;
+import static org.apache.datasketches.kll.KllPreambleUtil.DATA_START_ADR_SINGLE_ITEM;
+import static org.apache.datasketches.kll.KllPreambleUtil.DEFAULT_K;
+import static org.apache.datasketches.kll.KllPreambleUtil.DY_MIN_K_SHORT_ADR;
+import static org.apache.datasketches.kll.KllPreambleUtil.EMPTY_BIT_MASK;
+import static org.apache.datasketches.kll.KllPreambleUtil.FAMILY_BYTE_ADR;
+import static org.apache.datasketches.kll.KllPreambleUtil.FLAGS_BYTE_ADR;
+import static org.apache.datasketches.kll.KllPreambleUtil.K_SHORT_ADR;
+import static org.apache.datasketches.kll.KllPreambleUtil.LEVEL_ZERO_SORTED_BIT_MASK;
+import static org.apache.datasketches.kll.KllPreambleUtil.M_BYTE_ADR;
+import static org.apache.datasketches.kll.KllPreambleUtil.NUM_LEVELS_BYTE_ADR;
+import static org.apache.datasketches.kll.KllPreambleUtil.N_LONG_ADR;
+import static org.apache.datasketches.kll.KllPreambleUtil.PREAMBLE_INTS_BYTE_ADR;
+import static org.apache.datasketches.kll.KllPreambleUtil.PREAMBLE_INTS_EMPTY_SINGLE;
+import static org.apache.datasketches.kll.KllPreambleUtil.PREAMBLE_INTS_FLOAT;
+import static org.apache.datasketches.kll.KllPreambleUtil.SERIAL_VERSION_EMPTY_FULL;
+import static org.apache.datasketches.kll.KllPreambleUtil.SERIAL_VERSION_SINGLE;
+import static org.apache.datasketches.kll.KllPreambleUtil.SER_VER_BYTE_ADR;
+import static org.apache.datasketches.kll.KllPreambleUtil.SINGLE_ITEM_BIT_MASK;
+import static org.apache.datasketches.kll.KllPreambleUtil.UPDATABLE_BIT_MASK;
import java.util.Arrays;
import org.apache.datasketches.Family;
import org.apache.datasketches.SketchesArgumentException;
import org.apache.datasketches.Util;
-import org.apache.datasketches.kll.PreambleUtil.MemoryCheck;
+import org.apache.datasketches.kll.KllPreambleUtil.MemoryCheck;
import org.apache.datasketches.memory.Memory;
import org.apache.datasketches.memory.WritableMemory;
@@ -57,7 +57,7 @@ import org.apache.datasketches.memory.WritableMemory;
* Please refer to the documentation in the package-info:<br>
* {@link org.apache.datasketches.kll}
*/
-public class KllFloatsSketch extends HeapKllSketch {
+public class KllFloatsSketch extends KllHeapSketch {
// Specific to the floats sketch
private float[] items_; // the continuous array of float items
diff --git a/src/main/java/org/apache/datasketches/kll/HeapKllSketch.java b/src/main/java/org/apache/datasketches/kll/KllHeapSketch.java
similarity index 97%
rename from src/main/java/org/apache/datasketches/kll/HeapKllSketch.java
rename to src/main/java/org/apache/datasketches/kll/KllHeapSketch.java
index 28f5ce2..5d0c494 100644
--- a/src/main/java/org/apache/datasketches/kll/HeapKllSketch.java
+++ b/src/main/java/org/apache/datasketches/kll/KllHeapSketch.java
@@ -19,7 +19,7 @@
package org.apache.datasketches.kll;
-abstract class HeapKllSketch extends KllSketch {
+abstract class KllHeapSketch extends KllSketch {
/*
* Data is stored in items_.
@@ -48,7 +48,7 @@ abstract class HeapKllSketch extends KllSketch {
* Heap constructor.
* @param k configured size of sketch. Range [m, 2^16]
*/
- HeapKllSketch(final int k, final SketchType sketchType) {
+ KllHeapSketch(final int k, final SketchType sketchType) {
super(k, sketchType);
KllHelper.checkK(k);
dyMinK_ = k;
diff --git a/src/main/java/org/apache/datasketches/kll/KllHelper.java b/src/main/java/org/apache/datasketches/kll/KllHelper.java
index 8f286c7..4020883 100644
--- a/src/main/java/org/apache/datasketches/kll/KllHelper.java
+++ b/src/main/java/org/apache/datasketches/kll/KllHelper.java
@@ -21,8 +21,8 @@ package org.apache.datasketches.kll;
import static java.lang.Math.pow;
import static org.apache.datasketches.Util.floorPowerOf2;
-import static org.apache.datasketches.kll.PreambleUtil.MAX_K;
-import static org.apache.datasketches.kll.PreambleUtil.MIN_K;
+import static org.apache.datasketches.kll.KllPreambleUtil.MAX_K;
+import static org.apache.datasketches.kll.KllPreambleUtil.MIN_K;
import org.apache.datasketches.SketchesArgumentException;
import org.apache.datasketches.kll.KllSketch.SketchType;
diff --git a/src/main/java/org/apache/datasketches/kll/PreambleUtil.java b/src/main/java/org/apache/datasketches/kll/KllPreambleUtil.java
similarity index 99%
rename from src/main/java/org/apache/datasketches/kll/PreambleUtil.java
rename to src/main/java/org/apache/datasketches/kll/KllPreambleUtil.java
index b19d64a..c643927 100644
--- a/src/main/java/org/apache/datasketches/kll/PreambleUtil.java
+++ b/src/main/java/org/apache/datasketches/kll/KllPreambleUtil.java
@@ -85,9 +85,9 @@ import org.apache.datasketches.memory.WritableMemory;
*
* @author Lee Rhodes
*/
-final class PreambleUtil {
+final class KllPreambleUtil {
- private PreambleUtil() {}
+ private KllPreambleUtil() {}
static final String LS = System.getProperty("line.separator");
diff --git a/src/main/java/org/apache/datasketches/kll/KllSketch.java b/src/main/java/org/apache/datasketches/kll/KllSketch.java
index 0334c30..27b84b6 100644
--- a/src/main/java/org/apache/datasketches/kll/KllSketch.java
+++ b/src/main/java/org/apache/datasketches/kll/KllSketch.java
@@ -27,14 +27,14 @@ import static java.lang.Math.max;
import static java.lang.Math.min;
import static java.lang.Math.round;
import static org.apache.datasketches.kll.KllHelper.getAllLevelStatsGivenN;
+import static org.apache.datasketches.kll.KllPreambleUtil.DATA_START_ADR_DOUBLE;
+import static org.apache.datasketches.kll.KllPreambleUtil.DATA_START_ADR_FLOAT;
+import static org.apache.datasketches.kll.KllPreambleUtil.DATA_START_ADR_SINGLE_ITEM;
+import static org.apache.datasketches.kll.KllPreambleUtil.DEFAULT_M;
+import static org.apache.datasketches.kll.KllPreambleUtil.MAX_K;
+import static org.apache.datasketches.kll.KllPreambleUtil.MIN_K;
+import static org.apache.datasketches.kll.KllPreambleUtil.N_LONG_ADR;
import static org.apache.datasketches.kll.KllSketch.SketchType.DOUBLE_SKETCH;
-import static org.apache.datasketches.kll.PreambleUtil.DATA_START_ADR_DOUBLE;
-import static org.apache.datasketches.kll.PreambleUtil.DATA_START_ADR_FLOAT;
-import static org.apache.datasketches.kll.PreambleUtil.DATA_START_ADR_SINGLE_ITEM;
-import static org.apache.datasketches.kll.PreambleUtil.DEFAULT_M;
-import static org.apache.datasketches.kll.PreambleUtil.MAX_K;
-import static org.apache.datasketches.kll.PreambleUtil.MIN_K;
-import static org.apache.datasketches.kll.PreambleUtil.N_LONG_ADR;
import java.util.Random;
diff --git a/src/test/java/org/apache/datasketches/kll/KllDoublesSketchTest.java b/src/test/java/org/apache/datasketches/kll/KllDoublesSketchTest.java
index 20d7c37..ecbafc6 100644
--- a/src/test/java/org/apache/datasketches/kll/KllDoublesSketchTest.java
+++ b/src/test/java/org/apache/datasketches/kll/KllDoublesSketchTest.java
@@ -20,9 +20,9 @@
package org.apache.datasketches.kll;
//import static org.apache.datasketches.Util.getResourceBytes;
-import static org.apache.datasketches.kll.PreambleUtil.DEFAULT_K;
-import static org.apache.datasketches.kll.PreambleUtil.MAX_K;
-import static org.apache.datasketches.kll.PreambleUtil.MIN_K;
+import static org.apache.datasketches.kll.KllPreambleUtil.DEFAULT_K;
+import static org.apache.datasketches.kll.KllPreambleUtil.MAX_K;
+import static org.apache.datasketches.kll.KllPreambleUtil.MIN_K;
import static org.testng.Assert.assertEquals;
import static org.testng.Assert.assertFalse;
import static org.testng.Assert.assertNotNull;
@@ -160,23 +160,23 @@ public class KllDoublesSketchTest {
final KllDoublesSketch sketch2 = new KllDoublesSketch();
final int n = 10000;
for (int i = 0; i < n; i++) {
- sketch1.update(i);
- sketch2.update(2 * n - i - 1);
+ sketch1.update(i * 1.0);
+ sketch2.update((2 * n - i - 1) * 1.0);
}
- assertEquals(sketch1.getMinValue(), 0.0f);
- assertEquals(sketch1.getMaxValue(), n - 1f);
+ assertEquals(sketch1.getMinValue(), 0.0);
+ assertEquals(sketch1.getMaxValue(), (n - 1)*1.0);
- assertEquals(sketch2.getMinValue(), n);
- assertEquals(sketch2.getMaxValue(), 2f * n - 1f);
+ assertEquals(sketch2.getMinValue(), n * 1.0);
+ assertEquals(sketch2.getMaxValue(), (2 * n - 1) * 1.0);
sketch1.merge(sketch2);
assertFalse(sketch1.isEmpty());
assertEquals(sketch1.getN(), 2L * n);
- assertEquals(sketch1.getMinValue(), 0f);
- assertEquals(sketch1.getMaxValue(), 2f * n - 1);
- assertEquals(sketch1.getQuantile(0.5), n, n * PMF_EPS_FOR_K_256);
+ assertEquals(sketch1.getMinValue(), 0.0);
+ assertEquals(sketch1.getMaxValue(), (2 * n - 1) * 1.0);
+ assertEquals(sketch1.getQuantile(0.5), n * 1.0, n * PMF_EPS_FOR_K_256);
}
@Test
diff --git a/src/test/java/org/apache/datasketches/kll/KllFloatsSketchTest.java b/src/test/java/org/apache/datasketches/kll/KllFloatsSketchTest.java
index 3360fc9..87d992c 100644
--- a/src/test/java/org/apache/datasketches/kll/KllFloatsSketchTest.java
+++ b/src/test/java/org/apache/datasketches/kll/KllFloatsSketchTest.java
@@ -19,9 +19,9 @@
package org.apache.datasketches.kll;
-import static org.apache.datasketches.kll.PreambleUtil.DEFAULT_K;
-import static org.apache.datasketches.kll.PreambleUtil.MAX_K;
-import static org.apache.datasketches.kll.PreambleUtil.MIN_K;
+import static org.apache.datasketches.kll.KllPreambleUtil.DEFAULT_K;
+import static org.apache.datasketches.kll.KllPreambleUtil.MAX_K;
+import static org.apache.datasketches.kll.KllPreambleUtil.MIN_K;
import static org.apache.datasketches.Util.getResourceBytes;
import static org.testng.Assert.assertEquals;
import static org.testng.Assert.assertFalse;
@@ -160,23 +160,23 @@ public class KllFloatsSketchTest {
final KllFloatsSketch sketch2 = new KllFloatsSketch();
final int n = 10000;
for (int i = 0; i < n; i++) {
- sketch1.update(i);
- sketch2.update(2 * n - i - 1);
+ sketch1.update(i * 1.0f);
+ sketch2.update((2 * n - i - 1) * 1.0f);
}
assertEquals(sketch1.getMinValue(), 0.0f);
- assertEquals(sketch1.getMaxValue(), n - 1f);
+ assertEquals(sketch1.getMaxValue(), (n - 1) * 1.0f);
- assertEquals(sketch2.getMinValue(), n);
- assertEquals(sketch2.getMaxValue(), 2f * n - 1f);
+ assertEquals(sketch2.getMinValue(), n * 1.0f);
+ assertEquals(sketch2.getMaxValue(), (2 * n - 1) * 1.0f);
sketch1.merge(sketch2);
assertFalse(sketch1.isEmpty());
assertEquals(sketch1.getN(), 2L * n);
- assertEquals(sketch1.getMinValue(), 0f);
- assertEquals(sketch1.getMaxValue(), 2f * n - 1);
- assertEquals(sketch1.getQuantile(0.5), n, n * PMF_EPS_FOR_K_256);
+ assertEquals(sketch1.getMinValue(), 0.0f);
+ assertEquals(sketch1.getMaxValue(), (2 * n - 1) * 1.0f);
+ assertEquals(sketch1.getQuantile(0.5), n * 1.0f, n * PMF_EPS_FOR_K_256);
}
@Test
diff --git a/src/test/java/org/apache/datasketches/kll/KllHelperTest.java b/src/test/java/org/apache/datasketches/kll/KllHelperTest.java
index 489d722..40f02d7 100644
--- a/src/test/java/org/apache/datasketches/kll/KllHelperTest.java
+++ b/src/test/java/org/apache/datasketches/kll/KllHelperTest.java
@@ -55,7 +55,7 @@ public class KllHelperTest {
KllDoublesSketch sk = new KllDoublesSketch(200);
for (int i = 1; i <= 533; i++) { sk.update(i); }
int retained = sk.getNumRetained();
- int numLevels = ((HeapKllSketch)sk).getNumLevels();
+ int numLevels = ((KllHeapSketch)sk).getNumLevels();
println("NumLevels: " + numLevels);
println("NumRetained: " + retained);
diff --git a/src/test/java/org/apache/datasketches/kll/MiscDoublesTest.java b/src/test/java/org/apache/datasketches/kll/MiscDoublesTest.java
index ec96559..28998e3 100644
--- a/src/test/java/org/apache/datasketches/kll/MiscDoublesTest.java
+++ b/src/test/java/org/apache/datasketches/kll/MiscDoublesTest.java
@@ -19,7 +19,7 @@
package org.apache.datasketches.kll;
-import static org.apache.datasketches.kll.PreambleUtil.DEFAULT_K;
+import static org.apache.datasketches.kll.KllPreambleUtil.DEFAULT_K;
import static org.testng.Assert.assertEquals;
import static org.testng.Assert.assertTrue;
diff --git a/src/test/java/org/apache/datasketches/kll/MiscFloatsTest.java b/src/test/java/org/apache/datasketches/kll/MiscFloatsTest.java
index 7751b87..5f9ebbe 100644
--- a/src/test/java/org/apache/datasketches/kll/MiscFloatsTest.java
+++ b/src/test/java/org/apache/datasketches/kll/MiscFloatsTest.java
@@ -19,7 +19,7 @@
package org.apache.datasketches.kll;
-import static org.apache.datasketches.kll.PreambleUtil.DEFAULT_K;
+import static org.apache.datasketches.kll.KllPreambleUtil.DEFAULT_K;
import static org.testng.Assert.assertEquals;
import static org.testng.Assert.assertTrue;
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@datasketches.apache.org
For additional commands, e-mail: commits-help@datasketches.apache.org