You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "Michael Wall (JIRA)" <ji...@apache.org> on 2016/02/09 18:02:18 UTC

[jira] [Created] (ACCUMULO-4138) CompactCommand description is incorrect

Michael Wall created ACCUMULO-4138:
--------------------------------------

             Summary: CompactCommand description is incorrect
                 Key: ACCUMULO-4138
                 URL: https://issues.apache.org/jira/browse/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 
{code}
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 time)
  -?,--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
{code}

However, the --begin-row is not inclusive.  Here is a simple demonstration.
{code}
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
{code}

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
(v6.3.4#6332)