Small improvements to new AccumuloStore related code#4121
Small improvements to new AccumuloStore related code#4121DomGarguilo merged 2 commits intoapache:elasticityfrom
Conversation
|
Does this set up a merge conflict with #4126 ? Not necessarily a problem more of something to watch out for depending on the merge order. |
Yea looks like there is some overlap but shouldn't be too much of a process to resolve. |
cshannon
left a comment
cshannon
left a comment
There was a problem hiding this comment.
The changes look good, the only thing is the changes in AbstractFateStore exist in 2.1 and main already in ZooStore as that code was refactored in elasticity so we should probably also make the same changes there. I don't think changing things there first would merge forward easily with the refactor so we could just merge this PR and then create another PR to backport the changes to 2.1 and main and then merge an empty commit forward into elasticity
core/src/main/java/org/apache/accumulo/core/fate/accumulo/AccumuloStore.java
Show resolved
Hide resolved
core/src/main/java/org/apache/accumulo/core/fate/accumulo/AccumuloStore.java
Show resolved
Hide resolved
4 participants
No description provided.