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

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

bq. But using V3 is not mandatory for old style table/CF perm checking.
Same I was also thinking.. That is how we should make it..  This patch doing 
the simple check in AccessController which makes it need HFile V3
Andy, Pls feel free to take the issue with the fix you would like to have in 
AccessController.

> Need HFile version check in security coprocessors
> -------------------------------------------------
>
>                 Key: HBASE-10173
>                 URL: https://issues.apache.org/jira/browse/HBASE-10173
>             Project: HBase
>          Issue Type: Improvement
>          Components: security
>    Affects Versions: 0.98.0, 0.99.0
>            Reporter: Anoop Sam John
>            Assignee: Anoop Sam John
>            Priority: Critical
>             Fix For: 0.98.0, 0.99.0
>
>         Attachments: HBASE-10173.patch, HBASE-10173_partial.patch
>
>
> Cell level visibility labels are stored as cell tags. So HFile V3 is the 
> minimum version which can support this feature. Better to have a version 
> check in VisibilityController. Some one using this CP but with any HFile 
> version as V2, we can better throw error.



--
This message was sent by Atlassian JIRA
(v6.1.4#6159)

Reply via email to