You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "Christopher Tubbs (Jira)" <ji...@apache.org> on 2020/10/31 10:00:05 UTC
[jira] [Resolved] (ACCUMULO-3285) Remove static methods from
ZooUtil and make a factory for instances.
[ https://issues.apache.org/jira/browse/ACCUMULO-3285?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Christopher Tubbs resolved ACCUMULO-3285.
-----------------------------------------
Resolution: Not A Problem
This is OBE, as ZooUtil doesn't contain any of the retry code anymore, and all that is in ZooReader/ZooReaderWriter now. If there's more to be done here, please open a new issue or PR at https://github.com/apache/accumulo
> Remove static methods from ZooUtil and make a factory for instances.
> --------------------------------------------------------------------
>
> Key: ACCUMULO-3285
> URL: https://issues.apache.org/jira/browse/ACCUMULO-3285
> Project: Accumulo
> Issue Type: Improvement
> Reporter: Josh Elser
> Priority: Major
>
> 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).
--
This message was sent by Atlassian Jira
(v8.3.4#803005)