[jira] [Commented] (HBASE-11001) Shell support for granting cell permissions for testing

["ramkrishna.s.vasudevan (JIRA)" <ji...@apache.org>]

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

ramkrishna.s.vasudevan commented on HBASE-11001:
------------------------------------------------

Should we check for args[1] also here
{code}
if args[0].nil? or args[0].kind_of?(String)
{code}
because below we expect args[1] to be qualifier.
I would suggest make the *args to args = {} and for the case of 'grant' just do 
args[FAMILY], args[QUALIFIER].  So that the args are always of type Hash.  So need not check with 'arg[i].'



> Shell support for granting cell permissions for testing
> -------------------------------------------------------
>
>                 Key: HBASE-11001
>                 URL: https://issues.apache.org/jira/browse/HBASE-11001
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Andrew Purtell
>            Assignee: Andrew Purtell
>            Priority: Minor
>             Fix For: 0.99.0, 0.98.2
>
>         Attachments: 11001.patch
>
>
> For testing purposes it would be useful if the shell can support a simple syntax for adding cell ACLs to existing cells. Consider a combination of current 'grant' and 'scan' commands. 
> {noformat}
> grant '<userOrGroup>', '<perms>', '[<ns>:]<table>', { ... }
> {noformat}
> where the last argument is a scanner specification, for example (borrowed from scan.rb): \\
> - { COLUMNS => 'c1', STARTROW => 'abc', ENDROW => 'xyz' }
> - { COLUMNS => 'c1', TIMERANGE => [123, 456] }
> - { COLUMNS => ['c1', 'c2'], FILTER => "(PrefixFilter ('foo') AND
>     (QualifierFilter (=, 'bar:'))) AND (TimestampsFilter (123, 456))" }



--
This message was sent by Atlassian JIRA
(v6.2#6252)