leerho commented on PR #453:
URL:
https://github.com/apache/datasketches-java/pull/453#issuecomment-1590176116
Will noticed that when calling compact() on a Theta CompactSketch, instead
of just returning `this` the code created another compact object. After
digging into it, I realized that not only was the code not optimal, it was not
well documented either. The test code also needed to be rewritten to reflect
this updated behavior. This PR fixes these issues.
--
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]
