accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael Wall (JIRA)" <>
Subject [jira] [Created] (ACCUMULO-4138) CompactCommand description is incorrect
Date Tue, 09 Feb 2016 17:02:18 GMT
Michael Wall created ACCUMULO-4138:

             Summary: CompactCommand description is incorrect
                 Key: ACCUMULO-4138
             Project: Accumulo
          Issue Type: Bug
          Components: shell
    Affects Versions: 1.7.0, 1.6.4
            Reporter: Michael Wall
            Priority: Minor

The compact command has the following description 
root@accumulo> compact -?
usage: compact [<table>{ <table>}] [-?] [-b <begin-row>] [--cancel] [-e
<end-row>] [-nf] [-ns <namespace> | -p <pattern> | -t <tableName>]
 [-pn <profile>]  [-w]
description: sets all tablets for a table to major compact as soon as possible (based on current
  -?,--help                       display this help
  -b,--begin-row <begin-row>      begin row (inclusive)
     --cancel                     cancel user initiated compactions
  -e,--end-row <end-row>          end row (inclusive)
  -nf,--noFlush                   do not flush table data in memory before compacting.
  -ns,--namespace <namespace>     name of a namespace to operate on
  -p,--pattern <pattern>          regex pattern of table names to operate on
  -pn,--profile <profile>         iterator profile name
  -t,--table <tableName>          name of a table to operate on
  -w,--wait                       wait for compact to finish

However, the --begin-row is not inclusive.  Here is a simple demonstration.
createtable compacttest
addsplits a b c
insert "a" "1" "" ""
insert "a" "2" "" ""
insert "b" "3" "" ""
insert "b" "4" "" ""
insert "c" "5" "" ""
insert "c" "6" "" ""
flush -w
scan -t accumulo.metadata -np
compact -b a -e c -t compacttest -w
scan -t accumulo.metadata -np
deletetable compacttest -f

You will see that file associated with the 'a' split is still a F flush file, which the files
in the 'b' and 'c' split are A files.

Not sure if the fix is to update the commands description, which would be easy, or to make
the begin row actually inclusive.

This message was sent by Atlassian JIRA

View raw message