[ 
https://issues.apache.org/jira/browse/DRILL-6960?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16810198#comment-16810198
 ] 

ASF GitHub Bot commented on DRILL-6960:
---------------------------------------

kkhatua commented on issue #1608: DRILL-6960: AutoLimit the size of ResultSet 
for a WebUI (or REST) client
URL: https://github.com/apache/drill/pull/1608#issuecomment-480019695
 
 
   Closing this in favor of #1714 (DRILL-7048) which includes these fixes.
 
----------------------------------------------------------------
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:
[email protected]


> Auto Limit Wrapping should not apply to non-select query
> --------------------------------------------------------
>
>                 Key: DRILL-6960
>                 URL: https://issues.apache.org/jira/browse/DRILL-6960
>             Project: Apache Drill
>          Issue Type: Bug
>          Components: Web Server
>    Affects Versions: 1.16.0
>            Reporter: Kunal Khatua
>            Assignee: Kunal Khatua
>            Priority: Blocker
>              Labels: doc-impacting, user-experience
>             Fix For: 1.16.0
>
>
> [~IhorHuzenko] pointed out that DRILL-6050 can 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> 
> ... }}.
> The fix should involve a javascript check for all non-select queries and not 
> apply the LIMIT wrap for those queries.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to