[ACCUMULO-3448] ZooUtil throws exception even when retry succeeds

• Type: Bug
• Status: Resolved
• Priority: Blocker
• Resolution: Fixed
• Fix Version/s: 1.6.2, 1.7.0
• Component/s: zookeeper
• Assignee: Josh Elser
• Reporter: Christopher Tubbs
• Created: 22/Dec/14 19:17
• Updated: 23/Dec/14 00:31
• Resolved: 23/Dec/14 00:31

Description

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.

Issue Links

• is related to: ACCUMULO-3242 Consolidate ZK code WRT retries
• is related to: ACCUMULO-3285 Remove static methods from ZooUtil and make a factory for instances.