fx19880617 commented on issue #4994: Make sql as default query format. URL: https://github.com/apache/incubator-pinot/pull/4994#issuecomment-575843663 > > * Switch UI query format from pql to sql. > > * Fix data table presentation for sql response > > I think what we need is: > > 1. New Controller endpoint `/sql`, similar to `/pql`, which sets the right query options (see new `/query/sql` endpoint in PinotCLientRequest) > 2. Invoke `/sql` by default in query console. Have 1 checkbox, `PQL` to switch to `pql` endpoint. > > Any specific reason we need to continue to expose granular options groupByMode and responseFormat to the user? Here I'm still using pql endpoint as we haven't fully test out calcite sql parser. But the query response and group by behavior will follow sql standard.
---------------------------------------------------------------- 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
