[GitHub] [pinot] Jackie-Jiang commented on pull request #9832: [multistage] support sort push-down

[GitBox <gi...@apache.org>]

Jackie-Jiang commented on PR #9832:
URL: https://github.com/apache/pinot/pull/9832#issuecomment-1329930985

   Currently `serverReturnFinalResult` is used by v1 engine to return the final aggregation result instead of the intermediate aggregation result.
   The purpose for the flag in this PR is to order the columns based on the selection sequence (IIUC, the query constructed from v2 will always have order-by columns included in the selection sequence). This flag can also be applied to group-by queries which currently put group-by columns in from of other columns. We don't want to overload the same flag, so I'd suggest adding a separate flag for it.
   Alternatively, is it possible to construct the leaf stage query in a way that the v1 result always match the logical stage order?


-- 
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-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org