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

Anoop Sam John commented on HBASE-14122:
----------------------------------------

Patch looks great.
One question
{code}
/**
64         * Return true if cell authorization is supported and enabled
65         * @param connection The connection to use
66         * @return true if cell authorization is supported and enabled, false 
otherwise
67         * @throws IOException
68         */
69        public static boolean isCellAuthorizationSupported(Connection 
connection) throws IOException {
{code}
API checks whether the authorization is supported AND enabled.  The API name 
have to be XXXEnabled?   Current thing also ok. Just asking.

> Client API for determining if server side supports cell level security
> ----------------------------------------------------------------------
>
>                 Key: HBASE-14122
>                 URL: https://issues.apache.org/jira/browse/HBASE-14122
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Andrew Purtell
>            Priority: Minor
>             Fix For: 2.0.0, 0.98.14, 1.2.0, 1.3.0
>
>         Attachments: HBASE-14122.patch
>
>
> Add a client API for determining if the server side supports cell level 
> security. 
> Ask the master, assuming as we do in many other instances that the master and 
> regionservers all have a consistent view of site configuration.
> Return {{true}} if all features required for cell level security are present, 
> {{false}} otherwise, or throw {{UnsupportedOperationException}} if the master 
> does not have support for the RPC call.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to