From de24a45e6ab9533dabf84f32847d725a81b8bb66 Mon Sep 17 00:00:00 2001 From: Lee Rhodes Date: Thu, 21 Apr 2022 11:28:03 -0700 Subject: [PATCH] Remove unused method --- .../apache/datasketches/kll/KllDirectFloatsSketch.java | 8 -------- .../org/apache/datasketches/kll/KllDoublesSketch.java | 5 ----- .../org/apache/datasketches/kll/KllHeapFloatsSketch.java | 5 ----- src/main/java/org/apache/datasketches/kll/KllSketch.java | 2 -- 4 files changed, 20 deletions(-) diff --git a/src/main/java/org/apache/datasketches/kll/KllDirectFloatsSketch.java b/src/main/java/org/apache/datasketches/kll/KllDirectFloatsSketch.java index d6122bf9e..ec057f860 100644 --- a/src/main/java/org/apache/datasketches/kll/KllDirectFloatsSketch.java +++ b/src/main/java/org/apache/datasketches/kll/KllDirectFloatsSketch.java @@ -187,14 +187,6 @@ void setFloatItemsArrayAt(final int index, final float value) { wmem.putFloat(offset, value); } - @Override - void setFloatItemsArraySubrange(final float[] srcArr, final int srcIndex, final int tgtIndex, final int numItems) { - if (readOnly) { kllSketchThrow(TGT_IS_READ_ONLY); } - final int offset = - DATA_START_ADR + getLevelsArray().length * Integer.BYTES + (tgtIndex + 2) * Float.BYTES; - wmem.putFloatArray(offset, srcArr, srcIndex, numItems); - } - @Override void setLevelZeroSorted(final boolean sorted) { if (readOnly) { kllSketchThrow(TGT_IS_READ_ONLY); } diff --git a/src/main/java/org/apache/datasketches/kll/KllDoublesSketch.java b/src/main/java/org/apache/datasketches/kll/KllDoublesSketch.java index f9a35844a..6ef6a172e 100644 --- a/src/main/java/org/apache/datasketches/kll/KllDoublesSketch.java +++ b/src/main/java/org/apache/datasketches/kll/KllDoublesSketch.java @@ -356,11 +356,6 @@ public void update(final double value) { @Override //Artifact of inheritance void setFloatItemsArrayAt(final int index, final float value) { kllSketchThrow(MUST_NOT_CALL); } - @Override //Artifact of inheritance - void setFloatItemsArraySubrange(final float[] srcArr, final int srcIndex, final int tgtIndex, final int numItems) { - kllSketchThrow(MUST_NOT_CALL); - } - @Override //Artifact of inheritance void setMaxFloatValue(final float value) { kllSketchThrow(MUST_NOT_CALL); } diff --git a/src/main/java/org/apache/datasketches/kll/KllHeapFloatsSketch.java b/src/main/java/org/apache/datasketches/kll/KllHeapFloatsSketch.java index b671da6fd..3499850e6 100644 --- a/src/main/java/org/apache/datasketches/kll/KllHeapFloatsSketch.java +++ b/src/main/java/org/apache/datasketches/kll/KllHeapFloatsSketch.java @@ -172,11 +172,6 @@ void incNumLevels() { } //not used here @Override void setFloatItemsArrayAt(final int index, final float value) { floatItems_[index] = value; } - @Override - void setFloatItemsArraySubrange(final float[] srcArr, final int srcIndex, final int tgtIndex, final int numItems) { - System.arraycopy(srcArr, srcIndex, floatItems_, tgtIndex, numItems); - } - @Override void setLevelZeroSorted(final boolean sorted) { this.isLevelZeroSorted_ = sorted; } diff --git a/src/main/java/org/apache/datasketches/kll/KllSketch.java b/src/main/java/org/apache/datasketches/kll/KllSketch.java index 679b10675..896cb1b9d 100644 --- a/src/main/java/org/apache/datasketches/kll/KllSketch.java +++ b/src/main/java/org/apache/datasketches/kll/KllSketch.java @@ -506,8 +506,6 @@ final boolean isCompactSingleItem() { abstract void setFloatItemsArrayAt(int index, float value); - abstract void setFloatItemsArraySubrange(float[] srcArr, int srcIndex, int tgtIndex, int numItems); - final void setLevelsArray(final int[] levelsArr) { if (readOnly) { kllSketchThrow(TGT_IS_READ_ONLY); } this.levelsArr = levelsArr;