pgaref commented on pull request #651: URL: https://github.com/apache/orc/pull/651#issuecomment-842585252
> 10000 Hey @autumnust thanks for pinging me, got distracted by various tasks last week! Also thanks for updating the benchmark, this helps a lot. Some comments: - I would expect NONE dictImpl bench results to be identical across runs, why is 4096 X 500 run 3x the others? - Seems like an init size of 4k could be a good default conf in most of the cases -- would it make sense remove the extra conf completely if we can avoid it? - Minor: I would rename upperBound to distinctCount Let me know what you think -- 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]
