accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bill Havanki" <>
Subject Review Request 22399: ACCUMULO-2876 - use site config for default VolumeManagerImpl
Date Mon, 09 Jun 2014 20:47:15 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for accumulo, Eric Newton and Josh Elser.

Bugs: ACCUMULO-2876

Repository: accumulo


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.


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



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


Bill Havanki

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message