You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2020/11/12 12:43:35 UTC

[GitHub] [accumulo] jmark99 commented on pull request #1771: Add tablename option to shell insert command.

jmark99 commented on pull request #1771:
URL: https://github.com/apache/accumulo/pull/1771#issuecomment-726055015


   Hi Ed,
   
   I think that would be a good idea. Unless you wish to pursue it, I would be willing to take a look since I’m currently reviewing the Accumulo-examples code and coming across various commands that may need the option. Let me know which you would prefer.
   
   Mark
   
   
   From: EdColeman <no...@github.com>
   Sent: Tuesday, November 10, 2020 2:40 PM
   To: apache/accumulo <ac...@noreply.github.com>
   Cc: Mark Owens <jm...@apache.org>; State change <st...@noreply.github.com>
   Subject: Re: [apache/accumulo] Add tablename option to shell insert command. (#1771)
   
   
   I realize this has been merged - but I have a more general question. This is at least the third time that a table option has been added to a shell command (I added it for importdir, there was another, but I don't recall specifics) and then this one. Would it be worth creating an issue and checking that we are consistent across all of the commands that operate on a table that they accept this as an option?
   
   —
   You are receiving this because you modified the open/close state.
   Reply to this email directly, view it on GitHub<https://github.com/apache/accumulo/pull/1771#issuecomment-724923053>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AHQ4KYKDHGZBYOTEPXYBYSDSPGJIRANCNFSM4TM77QNQ>.
   


----------------------------------------------------------------
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:
users@infra.apache.org