[
https://issues.apache.org/jira/browse/ACCUMULO-4138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15151090#comment-15151090
]
Christopher Tubbs commented on ACCUMULO-4138:
---------------------------------------------
After discussing with [~kturner], I think maybe it's safest to update the
documentation, regardless of original intent or intuitiveness of the
exclusiveness of the start row, because some people may be relying on existing
behavior. This would also keep the behavior consistent with merge and other
tablet-centric operations (though inconsistent with scan-centric operations).
> 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.6.4, 1.7.0
> Reporter: Michael Wall
> Assignee: Michael Wall
> Labels: newbie
> Fix For: 1.6.6, 1.7.1, 1.8.0
>
>
> 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)
