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

CompactCommand description is incorrect

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 1.6.4, 1.7.0
    • 1.6.6, 1.7.1, 1.8.0
    • shell

    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

              mjwall Michael Wall
              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