jmark99 opened a new pull request #1797: URL: https://github.com/apache/accumulo/pull/1797
This ticket examined the table-related shell commands to examine if they provided a means to indicate the table name. The 'insert' command was updated recently to add the ability to supply a target table. This ticket provides the same capability for the 'delete' command. In all, there are 39 shell commands that can take a table name parameter. Each of the 39 now allow a table name to be supplied. For completeness, the commands are listed here: deleteiter, deletescaniter, listiter, setiter, setscaniter, grant, revoke, config, deletetable, droptable, du, exporttable, offline, online, summaries, addsplits, compact, constraint, flush, getgroups, getsplits, merge, setgroups, delete, deletemany, deleterows, egrep, formatter, interpreter, grep, importdirectory, insert, maxrow, scan, table, clonetable, createtable, importtable, and renametable. All but the last six listed above allow the table to be supplied using the -t/--table prefix. The final six do not use of the -t/--table prefix. There are eight that allow the table name to be supplied using both the -t/--table prefix or just the table name (deletetable, droptable, du, offline, online, summaries, compact, and flush). Would it be worth considering updating the final six to take the -t/--table prefix in order to provide consistency? They could mimic the eight that work with and without the prefix' in order to not break existing workflows. If there is support for doing so, I will create a follow-on ticket to continue the work. ---------------------------------------------------------------- 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]
