ddanielr commented on code in PR #6431:
URL: https://github.com/apache/accumulo/pull/6431#discussion_r3433110453
##########
shell/src/main/java/org/apache/accumulo/shell/commands/MergeCommand.java:
##########
@@ -52,6 +56,15 @@ public int execute(final String fullCommand, final
CommandLine cl, final Shell s
if (cl.hasOption(sizeOpt.getOpt())) {
size =
ConfigurationTypeHelper.getFixedMemoryAsBytes(cl.getOptionValue(sizeOpt.getOpt()));
}
+ if (tableName.equals(MetadataTable.NAME)) {
+ if (!shellState
+ .confirm(
+ " Warning!!! Merging the " + MetadataTable.NAME + " table
incorrectly can result in "
+ + "system instability. Are you REALLY sure you want to
merge?!?!?!")
Review Comment:
Added test cases for merge operations in MergeIT.
For the FateITs it sounds like you want to see what system interruptions
might occur if merging the metadata table happened while other fate operation
are ongoing.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
