[jira] [Commented] (ACCUMULO-589) Unable to set system properties in shell

["Billie Rinaldi (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/ACCUMULO-589?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13273634#comment-13273634 ] 

Billie Rinaldi commented on ACCUMULO-589:
-----------------------------------------

How do people feel about stability of shell command usage?  I am trying to decide what to do about this bug.  The problem is that config used the presence of the -t tableName flag to determine whether to use a table context or a system context.  Thus, config could not be made consistent with the other commands in terms of sensing the current table context and using that.  What I would really like to do is change config to use the new -a (-add) | -d (-delete) | -l (-list) set of flags (see the new ConstraintCommand), and then have -t <tableName> | -s (-system) flags to indicate the context (as grant and revoke do).  This sounds like it could make users unhappy, however.  I'll call this -a | -d | -l style of command an ADL command.  I hope to transition more of the commands to ADL options, anyway.

Another option for config would be to only allow the user to set system properties when outside of a table context.  This would still change the behavior, but less radically.  A third option would be to revert to the old config, deprecate it, and create a new ADL command with a different name.
                
> Unable to set system properties in shell
> ----------------------------------------
>
>                 Key: ACCUMULO-589
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-589
>             Project: Accumulo
>          Issue Type: Bug
>          Components: client
>            Reporter: Keith Turner
>            Assignee: Billie Rinaldi
>             Fix For: 1.5.0
>
>
> {noformat}
> root@test15> config -s master.tablet.balancer=org.apache.accumulo.server.master.balancer.TableLoadBalancer
> 11 12:35:08,206 [shell.Shell] ERROR: java.lang.IllegalStateException: Not in a table context. Please use 'table <tableName>' to switch to a table, or use '-t' to specify a table if option is available.
> root@test15> table
> table              tablepermissions   tables
> root@test15> table !METADATA
> root@test15 !METADATA> config -s master.tablet.balancer=org.apache.accumulo.server.master.balancer.TableLoadBalancer
> 11 12:35:31,581 [shell.Shell] ERROR: org.apache.accumulo.core.util.BadArgumentException: Invalid per-table property. near index 10
> config -s master.tablet.balancer=org.apache.accumulo.server.master.balancer.TableLoadBalancer
> {noformat}

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira