jmark99 opened a new pull request #409: ACCUMULO-4781 setshelliter command has 
incorrect usage statement
URL: https://github.com/apache/accumulo/pull/409
 
 
   The setshelliter command has incorrect usage statement. Update code to no 
longer require a table name in order for the command to execute.
   
   The setshelliter command allows a user to set an iterator profile that can 
be used within a shell session without having to continuously having to input 
all the iterator related arguments each time it is used. There is no need to 
use a table or namespace argument, yet one of those arguments were required in 
order to create a shell iterator. This was due to the fact that the 
setshelliter command inherited from both the setitercommand and shell classes, 
both of which require a table or namespace. 
   
   This update removes the need to supply the table/namespace arguments in 
order to create a shell iterator. Note that if the setshelliter command is used 
while within a table context the table argument was implied and the command 
would execute without issue. But if attempting to create a shell iterator when 
within the 'notable' context it would fail. To get around this issue, if the 
shell iterator command is called while not within a table context, the context 
is temporarily set to an existing table context and then reset once the shell 
iterator is created. 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to