Jackie-Jiang commented on PR #9078: URL: https://github.com/apache/pinot/pull/9078#issuecomment-1192057706
@siddharthteotia I understand the concern of breaking some prod queries, but IMO it is better to return exception rather than return unexpected result. The failed queries won't serve the purpose anyway, and returning an exception with proper error message is the best way to catch bad queries. Also, I don't think people have such queries, or they will complain when we added the `MinMaxValueBasedSelectionOrderByCombineOperator` -- 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
