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

(Updated Dec. 5, 2014, 9:13 p.m.)


Review request for accumulo.


Changes
-------

 * Added test for K suffix
 * added test for specifying compaction strat and file selection opt
 * fixed formatting
 * used enum instead of constants
 * consolidate test classes


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


Repository: accumulo


Description
-------

Added file selection and output configuration to shell compact command


Diffs (updated)
-----

  
core/src/main/java/org/apache/accumulo/core/compaction/CompactionSettings.java 
PRE-CREATION 
  
server/tserver/src/main/java/org/apache/accumulo/tserver/compaction/strategies/ConfigurableCompactionStrategy.java
 PRE-CREATION 
  shell/src/main/java/org/apache/accumulo/shell/Shell.java 8927ee0 
  shell/src/main/java/org/apache/accumulo/shell/commands/CompactCommand.java 
660630e 
  test/src/test/java/org/apache/accumulo/test/ShellServerIT.java d878c7f 

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


Testing
-------

Added an IT that test file selection options on compact command.  IT does not 
test output file configuration.


Thanks,

kturner

Reply via email to