himanshug commented on issue #9689: groupBy query: limit push down to segment scan is poor performance URL: https://github.com/apache/druid/issues/9689#issuecomment-613672659 > ... groupBy code to use Memory instead of ByteBuffer (#9308, #9314; see also MemoryBenchmark) partially motivated by performance. If we moved this part too it should help with initialization speed. thanks for pointing those out . from a quick look, it looks like we should just delete `ByteBufferHashTable` and replace all usage by `MemoryOpenHashTable` which is a clear winner. Do you see any reason for keeping `ByteBufferHashTable` around anywhere ? agree with everything, all of them could/should be independently done.
---------------------------------------------------------------- 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]
