Uploaded image for project: 'Accumulo'
  1. Accumulo
  2. ACCUMULO-4138

CompactCommand description is incorrect

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.6.4, 1.7.0
    • Fix Version/s: 1.6.6, 1.7.1, 1.8.0
    • Component/s: shell
    • Labels:

      Description

      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 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
      

      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.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                mjwall Michael Wall
                Reporter:
                mjwall Michael Wall
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 2h
                  2h