[jira] [Commented] (ACCUMULO-3448) ZooUtil throws exception even when retry succeeds

["Josh Elser (JIRA)" <ji...@apache.org>]

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

Josh Elser commented on ACCUMULO-3448:
--------------------------------------

Probably related to the changes that I made. We did not have any retry logic in released versions as far as I remember.

> ZooUtil throws exception even when retry succeeds
> -------------------------------------------------
>
>                 Key: ACCUMULO-3448
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-3448
>             Project: Accumulo
>          Issue Type: Bug
>          Components: fate
>            Reporter: Christopher Tubbs
>             Fix For: 1.7.0
>
>
> Saw this while examining checkstyle warnings:
> {{o.a.a.fate.zookeeper.ZooUtil}} has a static {{putData}} method which is called by other methods. In the {{OVERWRITE}} case, if an exception was caused by a timeout or other temporary, connection-related issue, it attempts to retry by calling a {{retryOrThrow}} method.
> However, even if it does end up succeeding (not throwing an exception), the case in the switch statement falls through to the default case, and throws the exception anyway, never retrying.
> May affect previous versions. I have not triage'd those yet.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)