clintropolis commented on issue #19344: URL: https://github.com/apache/druid/issues/19344#issuecomment-4270983627
>Store something in the metadata of segments to indicate which columns will not have any null values? We can then provide non-null implementations that fast-track to processing. We have this on a per segment level, `hasNulls` on the `ColumnCapabilities`. It probably wouldn't be that much work to have the non-vectorized topn check that for the column from selector factory when initializing the topN algorithm stuff. I would guess the aggregators are all probably also all checking isNull on their inputs which adds up, which I attempted to fix this once for simple numeric aggs for non-vectorized engines in #10666, though i never finished it and sort of forgot about it. -- 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]
