gianm commented on issue #6066: Sorting rows when rollup is disabled URL: https://github.com/apache/incubator-druid/issues/6066#issuecomment-410039394 IMO the speed hit on ingestion is acceptable. It is small, and performance is not really worse than the rollup case. Adding sorting will incur a performance cost no matter what: either it has to happen continuously or it has to happen in one big shot before persisting. Any idea where the speed hit on groupBys comes from? Seems strange that other query types wouldn't care but groupBy does. At any rate, I am not super worried about it, since in most clusters QueryableIndex storage/perf is much more serious issue than IncrementalIndex query perf. If we end up incurring a small speed hit on IncrementalIndex queries in order to get better compression on QueryableIndexes, then that is a good tradeoff.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org
