[ACCUMULO-3285] Remove static methods from ZooUtil and make a factory for instances.

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Not A Problem
• Assignee: Unassigned
• Reporter: Josh Elser
• Created: 31/Oct/14 16:36
• Updated: 31/Oct/20 09:59
• Resolved: 31/Oct/20 09:59

Description

In fixing ACCUMULO-2727, I wanted to write a unit test to be sure we properly retry when we get a BadVersion exception.

To get ZooKeeper to throw the exceptions I want it to, I really want to use EasyMock. However, to get mocking of static methods, I'd have to pull in PowerMock which I really don't want to do.

We should just take ZooUtil and wrap a factory around it to make some instances (or anything else that will ultimately remove the static methods).

Issue Links

• relates to: ACCUMULO-3448 ZooUtil throws exception even when retry succeeds