[
https://issues.apache.org/jira/browse/HIVE-10608?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14529147#comment-14529147
]
Chaoyu Tang commented on HIVE-10608:
------------------------------------
[~apivovarov] If so, I think the block should be like following by re-throwing
MetaException e for reasons rather than IOException and TTransportException:
{code}
} catch (MetaException e) {
if (e.getMessage().matches("(?s).*(IO|TTransport)Exception.*")) {
caughtException = e;
} else {
throw e;
}
}
{code}
> Fix useless 'if' stamement in RetryingMetaStoreClient (135)
> -----------------------------------------------------------
>
> Key: HIVE-10608
> URL: https://issues.apache.org/jira/browse/HIVE-10608
> Project: Hive
> Issue Type: Bug
> Components: Metastore
> Reporter: Alexander Pivovarov
> Assignee: Alexander Pivovarov
> Priority: Minor
>
> "if" statement below is useless because it ends with ;
> {code}
> } catch (MetaException e) {
> if (e.getMessage().matches("(?s).*(IO|TTransport)Exception.*"));
> caughtException = e;
> }
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
