Skip to content

Commit ce7650f

Browse files
authored
Fixed MergeCommand to use Option instead of String (apache#5456)
Prior to apache#5450 the all option in the merge command used an empty string for the short option. In 5450 we changed MergeCommand to use the Option.Builder, which allows specifying a long option only, but we didn't use the long option when getting the option value which causes a ShellServerIT failure. This changes the command line check to use the Option object instead of the option string, which is "", and the long option string.
1 parent bc8df24 commit ce7650f

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

shell/src/main/java/org/apache/accumulo/shell/commands/MergeCommand.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ public int execute(final String fullCommand, final CommandLine cl, final Shell s
4646
if (cl.hasOption(forceOpt.getOpt())) {
4747
force = true;
4848
}
49-
if (cl.hasOption(allOpt.getOpt())) {
49+
if (cl.hasOption(allOpt)) {
5050
all = true;
5151
}
5252
if (cl.hasOption(sizeOpt.getOpt())) {

0 commit comments

Comments
 (0)