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

Reply via email to