----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26507/ -----------------------------------------------------------
Review request for accumulo. Bugs: ACCUMULO-3177 and ACCUMULO-3178 https://issues.apache.org/jira/browse/ACCUMULO-3177 https://issues.apache.org/jira/browse/ACCUMULO-3178 Repository: accumulo Description ------- Added a per table volume chooser that allows tables to be given a specific volume chooser. The second patch, ACCUMULO-3178, adds an example, a preferred volume chooser which gives the preferred volume for a table. When a table chooser is not specified, or a preferred volume is not specified then, the default chooser is the RandomVolumeChooser. Diffs ----- core/src/main/java/org/apache/accumulo/core/conf/Property.java ad4fe92 server/base/src/main/java/org/apache/accumulo/server/fs/PerTableVolumeChooser.java PRE-CREATION server/base/src/main/java/org/apache/accumulo/server/fs/RandomVolumeChooser.java 2760b07 server/base/src/main/java/org/apache/accumulo/server/fs/VolumeChooser.java 8713c97 server/base/src/main/java/org/apache/accumulo/server/fs/VolumeChooserEnvironment.java PRE-CREATION server/base/src/main/java/org/apache/accumulo/server/fs/VolumeManager.java cbfdb5e server/base/src/main/java/org/apache/accumulo/server/fs/VolumeManagerImpl.java b2341b6 server/base/src/main/java/org/apache/accumulo/server/fs/VolumeUtil.java 82b77ee server/base/src/main/java/org/apache/accumulo/server/init/Initialize.java 4305e71 server/base/src/main/java/org/apache/accumulo/server/util/FileUtil.java 0f7ac22 server/base/src/main/java/org/apache/accumulo/server/util/MetadataTableUtil.java 8d14cad server/base/src/main/java/org/apache/accumulo/server/util/TabletOperations.java b8e7113 server/master/src/main/java/org/apache/accumulo/master/Master.java b435b0f server/master/src/main/java/org/apache/accumulo/master/TabletGroupWatcher.java 0a3d1d0 server/master/src/main/java/org/apache/accumulo/master/tableOps/CreateTable.java f56c1e2 server/tserver/src/main/java/org/apache/accumulo/tserver/log/DfsLogger.java e923ebc server/tserver/src/test/java/org/apache/accumulo/tserver/TabletServerSyncCheckTest.java dad9a75 Diff: https://reviews.apache.org/r/26507/diff/ Testing ------- New IT in the patch for ACCUMULO-3178. Could not test ACCUMULO-3177 without an example chooser. Thanks, Jenna Huston