You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@drill.apache.org by GitBox <gi...@apache.org> on 2019/01/09 10:57:15 UTC

[GitHub] ihuzenko commented on issue #1593: DRILL-6050: Provide a limit to number of rows fetched for a query in UI

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


With regards,
Apache Git Services