Jackie-Jiang commented on issue #4362: Optimize MutableOffHeapByteArrayStore by directly calling the PinotDataBuffer API URL: https://github.com/apache/incubator-pinot/pull/4362#issuecomment-506489793 @buchireddy Yes, the unit of the score is ms/op. You are correct that batch read/write will create a buffer duplicate, which is extra garbage. And that is why we put a threshold so that small read/write don't create the duplicate as an optimization (the old implementation always create the buffer). The duplicate of the buffer is always short-lived tiny garbage, which should have negligible GC impact. The benchmark is testing the read/write for hundreds millions times, so the GC should already be included.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
