[ACCUMULO-4516] TeraSortIngest splits argument is ignored if less than 10 arguments are provided

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.7.4, 1.9.0
• Component/s: examples
• Labels: newbie pull-request-available
• Assignee: Mark Owens
• Reporter: Josh Elser
• Created: 08/Nov/16 19:54
• Updated: 23/Apr/19 16:13
• Resolved: 23/Apr/19 16:07

Description

I invoked the TeraSortIngest and did not see my table being pre-split:

tool.sh lib/accumulo-examples-simple.jar org.apache.accumulo.examples.simple.mapreduce.TeraSortIngest -i accumulo  -z jelser-accumulo-scripts-1 -u root -p secret  --count $((1000 * 1000 * 50)) --minKeySiz
e 10 --maxKeySize 10 --minValueSize 78  --maxValueSize 78  --table terasort --splits 15

Turns out that the positional argument parsing got lost in the switch to jcommander:

    if (args.length > 10)
      conf.setInt(NUMSPLITS, opts.splits);

We should just set this value if the value for splits is not the default (0).

Issue Links

• links to: GitHub Pull Request #6