accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ke...@deenlo.com
Subject Re: Review Request 17863: ACCUMULO-1832 add operation replace volumes
Date Wed, 12 Feb 2014 02:42:00 GMT

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

(Updated Feb. 12, 2014, 2:41 a.m.)


Review request for accumulo.


Changes
-------

 * Switched back to using comma separated URIs.
 * Reformatted some files
 * Fixed a cut and paste bug.


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


Repository: accumulo


Description
-------

Add operation to replace volumes


Diffs (updated)
-----

  core/src/main/java/org/apache/accumulo/core/conf/Property.java bc4c7e1 
  core/src/main/java/org/apache/accumulo/core/tabletserver/log/LogEntry.java fc2da4b 
  server/base/src/main/java/org/apache/accumulo/server/ServerConstants.java 9d490e4 
  server/base/src/main/java/org/apache/accumulo/server/fs/VolumeManager.java c2c04e5 
  server/base/src/main/java/org/apache/accumulo/server/fs/VolumeManagerImpl.java b577891 
  server/base/src/main/java/org/apache/accumulo/server/fs/VolumeUtil.java PRE-CREATION 
  server/base/src/main/java/org/apache/accumulo/server/master/recovery/HadoopLogCloser.java
0006bf9 
  server/base/src/main/java/org/apache/accumulo/server/util/Admin.java 1e1bc79 
  server/base/src/main/java/org/apache/accumulo/server/util/ListVolumesUsed.java PRE-CREATION

  server/base/src/main/java/org/apache/accumulo/server/util/MetadataTableUtil.java 8b8a3d6

  server/base/src/test/java/org/apache/accumulo/server/fs/FileTypeTest.java PRE-CREATION 
  server/base/src/test/java/org/apache/accumulo/server/fs/VolumeUtilTest.java PRE-CREATION

  server/gc/src/main/java/org/apache/accumulo/gc/SimpleGarbageCollector.java 03519ba 
  server/master/src/main/java/org/apache/accumulo/master/recovery/RecoveryManager.java 2479d63

  server/tserver/src/main/java/org/apache/accumulo/tserver/DirectoryDecommissioner.java ea932ff

  server/tserver/src/main/java/org/apache/accumulo/tserver/Tablet.java dd2afad 
  server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServer.java 080cc20 
  server/tserver/src/test/java/org/apache/accumulo/tserver/DirectoryDecommissionerTest.java
47cdab9 
  server/tserver/src/test/java/org/apache/accumulo/tserver/TabletServerSyncCheckTest.java
590945a 
  test/src/test/java/org/apache/accumulo/test/CleanWalIT.java ed04f7e 
  test/src/test/java/org/apache/accumulo/test/ConfigurableMajorCompactionIT.java 28d183c 
  test/src/test/java/org/apache/accumulo/test/DumpConfigIT.java b98d452 
  test/src/test/java/org/apache/accumulo/test/MasterRepairsDualAssignmentIT.java afd1403 
  test/src/test/java/org/apache/accumulo/test/NoMutationRecoveryIT.java 4656d30 
  test/src/test/java/org/apache/accumulo/test/VolumeIT.java ee38efd 
  test/src/test/java/org/apache/accumulo/test/functional/BalanceAfterCommsFailureIT.java 08242ff

  test/src/test/java/org/apache/accumulo/test/functional/BatchScanSplitIT.java faf8777 
  test/src/test/java/org/apache/accumulo/test/functional/BigRootTabletIT.java 16130bf 
  test/src/test/java/org/apache/accumulo/test/functional/BinaryStressIT.java 4a10a34 
  test/src/test/java/org/apache/accumulo/test/functional/BloomFilterIT.java e3914a1 
  test/src/test/java/org/apache/accumulo/test/functional/BulkSplitOptimizationIT.java 337bd4f

  test/src/test/java/org/apache/accumulo/test/functional/ChaoticBalancerIT.java 9a4a904 
  test/src/test/java/org/apache/accumulo/test/functional/CleanTmpIT.java df1b51b 
  test/src/test/java/org/apache/accumulo/test/functional/CompactionIT.java 3f8fcb3 
  test/src/test/java/org/apache/accumulo/test/functional/ConcurrencyIT.java c09216f 
  test/src/test/java/org/apache/accumulo/test/functional/ConfigurableCompactionIT.java cc43cc9

  test/src/test/java/org/apache/accumulo/test/functional/ConfigurableMacIT.java 0de01c9 
  test/src/test/java/org/apache/accumulo/test/functional/DeleteEverythingIT.java f00a445 
  test/src/test/java/org/apache/accumulo/test/functional/DynamicThreadPoolsIT.java e83baee

  test/src/test/java/org/apache/accumulo/test/functional/GarbageCollectorIT.java 81377bc 
  test/src/test/java/org/apache/accumulo/test/functional/HalfDeadTServerIT.java d8f38c8 
  test/src/test/java/org/apache/accumulo/test/functional/LargeRowIT.java c311413 
  test/src/test/java/org/apache/accumulo/test/functional/LateLastContactIT.java 7f1f29b 
  test/src/test/java/org/apache/accumulo/test/functional/MasterFailoverIT.java 1fbe2ff 
  test/src/test/java/org/apache/accumulo/test/functional/MaxOpenIT.java 9483ab9 
  test/src/test/java/org/apache/accumulo/test/functional/MetadataMaxFiles.java 789d84b 
  test/src/test/java/org/apache/accumulo/test/functional/MetadataSplitIT.java bb0dd15 
  test/src/test/java/org/apache/accumulo/test/functional/RestartIT.java 709aa1d 
  test/src/test/java/org/apache/accumulo/test/functional/RestartStressIT.java da5a86f 
  test/src/test/java/org/apache/accumulo/test/functional/RowDeleteIT.java 0ee903b 
  test/src/test/java/org/apache/accumulo/test/functional/ScanSessionTimeOutIT.java b223845

  test/src/test/java/org/apache/accumulo/test/functional/SimpleBalancerFairnessIT.java fcb19fd

  test/src/test/java/org/apache/accumulo/test/functional/SplitIT.java 80b79ff 
  test/src/test/java/org/apache/accumulo/test/functional/SslIT.java 447f8ef 
  test/src/test/java/org/apache/accumulo/test/functional/SslWithClientAuthIT.java 79b3255

  test/src/test/java/org/apache/accumulo/test/functional/TabletIT.java d86742c 
  test/src/test/java/org/apache/accumulo/test/functional/WriteAheadLogIT.java 25d0757 
  test/src/test/java/org/apache/accumulo/test/functional/ZookeeperRestartIT.java d944778 

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


Testing
-------

Added unit test and IT.  VolumeIT is the only IT that needs review, all others were changed
because I changed a method signature


Thanks,

kturner


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