Changes responding to Copilot issues.

Author: leerho

src/main/java/org/apache/datasketches/quantiles/DirectUpdateDoublesSketch.java

@@ -217,7 +217,7 @@ public void reset() {
   //Restricted overrides
 
   @Override
-  HeapUpdateDoublesSketch getSketchAndReset() {
+  UpdateDoublesSketch getSketchAndReset() {
     final HeapUpdateDoublesSketch skCopy = heapify(this);
     reset();
     return skCopy;

src/main/java/org/apache/datasketches/quantiles/DirectUpdateDoublesSketchR.java

@@ -130,12 +130,12 @@ public boolean isSameResource(final MemorySegment that) {
 
   @Override
   public void reset() {
-    throw new SketchesReadOnlyException("Call to reset() on read-only buffer");
+    throw new SketchesReadOnlyException("Call to reset() on read-only sketch");
   }
 
   @Override
   public void update(final double dataItem) {
-    throw new SketchesReadOnlyException("Call to update() on read-only buffer");
+    throw new SketchesReadOnlyException("Call to update() on read-only sketch");
   }
 
   //Restricted overrides
@@ -147,8 +147,10 @@ int getBaseBufferCount() {
   }
 
   @Override
-  int getCombinedBufferItemCapacity() {
-    return Math.max(0, (int)seg_.byteSize() - COMBINED_BUFFER) / 8;
+  long getBitPattern() {
+    final int k = getK();
+    final long n = getN();
+    return ClassicUtil.computeBitPattern(k, n);
   }
 
   @Override
@@ -159,15 +161,12 @@ int getCombinedBufferItemCapacity() {
     final int itemCap = ClassicUtil.computeCombinedBufferItemCapacity(k, n);
     final double[] combinedBuffer = new double[itemCap];
     MemorySegment.copy(seg_, JAVA_DOUBLE_UNALIGNED, COMBINED_BUFFER, combinedBuffer, 0, itemCap);
-
     return combinedBuffer;
   }
 
   @Override
-  long getBitPattern() {
-    final int k = getK();
-    final long n = getN();
-    return ClassicUtil.computeBitPattern(k, n);
+  int getCombinedBufferItemCapacity() {
+    return Math.max(0, (int)seg_.byteSize() - COMBINED_BUFFER) / 8;
   }
 
   @Override
@@ -177,19 +176,24 @@ MemorySegment getMemorySegment() {
 
   @Override
   UpdateDoublesSketch getSketchAndReset() {
-    throw new SketchesReadOnlyException("Call to getResultAndReset() on read-only sketch");
+    throw new SketchesReadOnlyException("Call to getSketchAndReset() on read-only sketch");
+  }
+
+  @Override
+  double[] growCombinedBuffer(final int curCombBufItemCap, final int itemSpaceNeeded) {
+    throw new SketchesReadOnlyException("Call to growCombinedBuffer() on read-only sketch");
   }
 
   //Puts
 
   @Override
   void putMinItem(final double minQuantile) {
-    throw new SketchesReadOnlyException("Call to putMinQuantile() on read-only sketch");
+    throw new SketchesReadOnlyException("Call to putMinItem() on read-only sketch");
   }
 
   @Override
   void putMaxItem(final double maxQuantile) {
-    throw new SketchesReadOnlyException("Call to putMaxQuantile() on read-only sketch");
+    throw new SketchesReadOnlyException("Call to putMaxItem() on read-only sketch");
   }
 
   @Override
@@ -209,12 +213,7 @@ void putBaseBufferCount(final int baseBufferCount) {
 
   @Override
   void putBitPattern(final long bitPattern) {
-    throw new SketchesReadOnlyException("Call to putBaseBufferCount() on read-only sketch");
-  }
-
-  @Override
-  double[] growCombinedBuffer(final int curCombBufItemCap, final int itemSpaceNeeded) {
-    throw new SketchesReadOnlyException("Call to growCombinedBuffer() on read-only sketch");
+    throw new SketchesReadOnlyException("Call to putBitPattern() on read-only sketch");
   }
 
   //Checks

src/main/java/org/apache/datasketches/quantiles/ItemsUnion.java

@@ -182,7 +182,7 @@ public ItemsSketch<T> getResult() {
    *
    * @return the result of this Union operation and reset.
    */
-  public ItemsSketch<T> getResultAndReset() { //TODO
+  public ItemsSketch<T> getResultAndReset() {
     if (gadget_ == null) { return null; } //Intentionally return null here for speed.
     return gadget_.getSketchAndReset();
   }