-----------------------------------------------------------
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