Re: [PR] Improvements realized while analyzing Issue 514 (datasketches-java)

["jmalkin (via GitHub)" <gi...@apache.org>]

jmalkin commented on PR #516:
URL: https://github.com/apache/datasketches-java/pull/516#issuecomment-1975908971

   I'm not sure about the assert. Aren't those removed for non-debug builds? Meaning the use cases where we'd expect the asserts to possibly trigger are likely to be the situations where the assert is removed? I don't think we'd expect people to insert that many items into a debug-mode test.
   
   The changes are fine if we agree it's ok not to have those check in productions systems.


-- 
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: commits-unsubscribe@datasketches.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@datasketches.apache.org
For additional commands, e-mail: commits-help@datasketches.apache.org