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

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

kkhatua commented on pull request #1742: DRILL-7160: e.q.max_rows QUERY-level 
option shown even if not set
URL: https://github.com/apache/drill/pull/1742
 
 
   The fix is to force setting to zero IFF autoLimit was intended to be set 
originally but is inapplicable; such as 'SHOW DATABASES'. If autolimit was not 
intended to be applied, setting the value to zero is not required.
 
----------------------------------------------------------------
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]


> exec.query.max_rows QUERY-level options are shown on Profiles tab
> -----------------------------------------------------------------
>
>                 Key: DRILL-7160
>                 URL: https://issues.apache.org/jira/browse/DRILL-7160
>             Project: Apache Drill
>          Issue Type: Bug
>          Components: Web Server
>    Affects Versions: 1.16.0
>            Reporter: Volodymyr Vysotskyi
>            Assignee: Kunal Khatua
>            Priority: Blocker
>             Fix For: 1.16.0
>
>
> As [~arina] has noticed, option {{exec.query.max_rows}} is shown on Web UI's 
> Profiles even when it was not set explicitly. The issue is because the option 
> is being set on the query level internally.
> From the code, looks like it is set in 
> {{DrillSqlWorker.checkAndApplyAutoLimit()}}, and perhaps a check whether the 
> value differs from the existing one should be added.



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

Reply via email to