fx19880617 commented on issue #5005: Calcite SQL compiler syntax support for order by expression and set LENIENT SQL conformance URL: https://github.com/apache/incubator-pinot/pull/5005#issuecomment-577945931 > This LGTM. > I see you've also included the change to use LIMIT in group by queries. Please call that out as well in the description. sure, I think also for SQL parser, we actually don't support TOP, so LIMIT is used.
---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org