imply-cheddar commented on issue #12261: URL: https://github.com/apache/druid/issues/12261#issuecomment-1041343764
The value is currently being used to estimate the total size in memory of the onheap Aggregator object, so the semantics that you mentioned are exactly what we want. It's never used to determine or allocate the number of bytes actually required to store, persist or transport the sketch. It's also perfectly fine that it doesn't always produce a new value. That's understood and accounted for. Thanks for such a quick turnaround on an update to the base library. As soon as that's release, let us know and we will update the code to remove the reflection. -- 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]
