gortiz commented on PR #8979: URL: https://github.com/apache/pinot/pull/8979#issuecomment-1250593801
> Seems like a great optimization. Any plans of resuming this ? If I remember correctly there are two things we would need to improve: - Disable the optimization when null handling is active - Decide whether we should remove the descending optimization or not. As suggested by @Jackie-Jiang, it would make more sense (and be more efficient) to have descending operators. The current solution in this PR runs faster than the previous code, but given the complexities of the code, it is difficult to be sure whether the solution is always correct or not. -- 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: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
