[GitHub] [druid] vogievetsky opened a new issue #11212: DruidSQL should allow (and trim out) trailing semicolons;

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

vogievetsky opened a new issue #11212:
URL: https://github.com/apache/druid/issues/11212


   ### Description
   
   It is pretty standard to finish queries with semicolons, right now this fails to parse in Druid. It would be really easy to trim out the trailing semicolons and ignore them just like comments are ignored
   
   ### Motivation
   
   It would improve the query UX just a little bit.
   
   Right now
   
   `curl 'http://localhost:8888/druid/v2/sql' -H 'Content-Type: application/json' --data-raw '{"query":"SELECT isNew, COUNT(*) FROM wikipedia GROUP BY 1 ORDER BY 2 DESC;"}'`  
   
   Fails with
   
   `{"error":"SQL parse failed","errorMessage":"Encountered \";\" at line 1, column 65.\nWas expecting one of:\n    <EOF> \n    \"FETCH\" ...\n    \"LIMIT\" ...\n    \"NULLS\" ...\n    \"OFFSET\" ...\n    \",\" ...\n    ","errorClass":"org.apache.calcite.sql.parser.SqlParseException","host":null}`


-- 
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:
users@infra.apache.org



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