jmark99 commented on a change in pull request #1771:
URL: https://github.com/apache/accumulo/pull/1771#discussion_r520115394
##########
File path:
shell/src/main/java/org/apache/accumulo/shell/commands/InsertCommand.java
##########
@@ -171,6 +180,11 @@ public Options getOptions() {
"durability to use for insert, should be one of \"none\" \"log\"
\"flush\" or \"sync\"");
o.addOption(durabilityOption);
+ tableNameOption =
+ new Option("", "tablename", true, "name of table in which data is
being inserted");
+ tableNameOption.setArgName("tablename");
+ o.addOption(tableNameOption);
+
Review comment:
I chose not to use the short form to minimize any confusion with the
<code>timestamp</code> parameter (-ts). There is also a <code>timeout<code>
parameter that does not off a shortened name. If you think it would be nice to
include one, I have no problem adding one. Could always use '-n' for 'name'
perhaps. Let me know your thoughts.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
