siddharthteotia commented on issue #4877: Results in ResultTable if responseFormat=sql URL: https://github.com/apache/incubator-pinot/pull/4877#issuecomment-561857097 LGTM. One high level comment I think it is fine to have a PQL flavor in SQL group by execution mode -- thus giving users to get results in both SQL (ResultTable) and PQL (for backward compatibility) However, for PQL group by execution mode, we should probably consider not having any SQL flavor there. Currently we still allow the results to be in SQL if there is exactly one aggregation function and responseFormat is SQL even though the execution mode was PQL. If the user chose to use PQL, then everything (response, execution) should be PQL style.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
