jmalkin commented on code in PR #567:
URL: https://github.com/apache/datasketches-java/pull/567#discussion_r1622807984
##########
src/main/java/org/apache/datasketches/theta/Sketches.java:
##########
@@ -79,15 +79,32 @@ public static int getMaxAnotBResultBytes(final int
maxNomEntries) {
}
/**
- * Ref: {@link Sketch#getMaxCompactSketchBytes(int)}
- * @param numberOfEntries Ref: {@link Sketch#getMaxCompactSketchBytes(int)},
- * {@code numberOfEntries}
- * @return Ref: {@link Sketch#getMaxCompactSketchBytes(int)}
- */
+ * Returns the maximum number of storage bytes required for a CompactSketch
with the given
+ * number of actual entries. Note that this assumes the worse case of the
sketch in
Review Comment:
Didn't even mean to pull this in. I think I inadvertently pulled form the
main branch -- I thought I fetched and switches to the quantiles-filter branch!
But will fix.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
