accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jenna Huston" <jenna.husto...@gmail.com>
Subject Re: Review Request 26507: ACCUMULO-3177 Create a per table volume chooser and ACCUMULO-3178 Create example preferred volumes chooser
Date Mon, 27 Oct 2014 20:13:53 GMT

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

(Updated Oct. 27, 2014, 8:13 p.m.)


Review request for accumulo.


Changes
-------

Changed code to not depend on ACCUMULO-3176.  Used namespace properties instead.


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 (updated)
-----

  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 e4a42dc 
  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 602f214 
  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 5f34279

  server/base/src/main/java/org/apache/accumulo/server/util/TabletOperations.java b8e7113

  server/master/src/main/java/org/apache/accumulo/master/Master.java 42495f4 
  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/master/src/main/java/org/apache/accumulo/master/tableOps/ImportTable.java 44ca898

  server/master/src/test/java/org/apache/accumulo/master/tableOps/ImportTableTest.java 31f6bde

  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.


File Attachments
----------------

Diff for 3178
  https://reviews.apache.org/media/uploaded/files/2014/10/09/07d2693e-9acc-438b-9b13-667bde467590__0001-ACCUMULO-3178-Create-example-preferred-volumes-choos.patch
Updated Diff for 3178
  https://reviews.apache.org/media/uploaded/files/2014/10/14/2b5aa6b9-92d0-4938-bcf9-2021cd0212ee__0001-ACCUMULO-3178-Create-example-preferred-volumes-choos.patch


Thanks,

Jenna Huston


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