You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hive.apache.org by "Zoltan Haindrich (JIRA)" <ji...@apache.org> on 2018/02/05 13:26:01 UTC

[jira] [Updated] (HIVE-17991) Remove CommandNeedRetryException

     [ https://issues.apache.org/jira/browse/HIVE-17991?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Zoltan Haindrich updated HIVE-17991:
------------------------------------
    Attachment: HIVE-17991.02.patch

> Remove CommandNeedRetryException
> --------------------------------
>
>                 Key: HIVE-17991
>                 URL: https://issues.apache.org/jira/browse/HIVE-17991
>             Project: Hive
>          Issue Type: Sub-task
>            Reporter: Zoltan Haindrich
>            Assignee: Zoltan Haindrich
>            Priority: Major
>         Attachments: HIVE-17991.01.patch, HIVE-17991.02.patch
>
>
> * only really used in clidriver (via an infinite loop) [here|https://github.com/apache/hive/blob/master/cli/src/java/org/apache/hadoop/hive/cli/CliDriver.java#L307]
> * only 1 thing depends on the value of trycount; I think we can remove it  [GlobalLimitOptimizer|https://github.com/apache/hive/blob/master/ql/src/java/org/apache/hadoop/hive/ql/optimizer/GlobalLimitOptimizer.java#L78]
> * there are a few loops which have pre-set trycount to 1 [example|https://github.com/apache/hive/blob/master/hcatalog/streaming/src/java/org/apache/hive/hcatalog/streaming/HiveEndPoint.java#L497]
> * other catches are exception repackagings
> I think it would be better to remove it.
> for more info:
> {code}
> git grep -C10 'catch.*CommandNeedRetryException'
> git grep -i gettrycount
> {code}



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