-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22399/
-----------------------------------------------------------

Review request for accumulo, Eric Newton and Josh Elser.


Bugs: ACCUMULO-2876
    https://issues.apache.org/jira/browse/ACCUMULO-2876


Repository: accumulo


Description
-------

This changes VolumeManagerImpl.get() to use the site configuration instead of 
the ZooKeeper-based system configuration. This seems to align better with what 
is intended, and it also eliminates the risk of an infinite loop, where the 
system configuration requires an HdfsZooInstance instance ID, which requires a 
volume manager.


Diffs
-----

  
server/base/src/main/java/org/apache/accumulo/server/fs/VolumeManagerImpl.java 
8fe6579 

Diff: https://reviews.apache.org/r/22399/diff/


Testing
-------

All unit tests pass. The fix also rectified the loop in SystemCredentialsTest 
after refactoring for ACCUMULO-2615.


Thanks,

Bill Havanki

Reply via email to