ihuzenko commented on issue #1593: DRILL-6050: Provide a limit to number of 
rows fetched for a query in UI
URL: https://github.com/apache/drill/pull/1593#issuecomment-452656286
 
 
   Hello @kkhatua , this solution may cause submission of queries with 
incorrect syntax. For example, when user enters 'SHOW DATABASES' and after 
limitation wrapping 'SELECT * FROM (SHOW DATABASES) LIMIT 10' will be posted. 
This results into parsing errors, like: 
   `Query Failed: An Error Occurred
   org.apache.drill.common.exceptions.UserRemoteException: PARSE ERROR: 
Encountered "( show" at line 2, column 15. Was expecting one of: <IDENTIFIER> 
... `. 
   
   Could you please fix the issue and test limit functionality with all 
non-select kinds of queries supported by Drill ? Also I'd like to kindly ask 
you for change tooltip for question mark from onclick alert to on hover one, 
for example see [Tooltips](https://www.w3schools.com/css/css_tooltip.asp). 
Thank you in advance. 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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

Reply via email to