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

stack commented on HBASE-12663:
-------------------------------

Failure is HBASE-12664

Patch LGTM. +1  Safe to add in 1.0 because it is additive, not removing 
anything.


> unify getTableDescriptors() and listTableDescriptorsByNamespace()
> -----------------------------------------------------------------
>
>                 Key: HBASE-12663
>                 URL: https://issues.apache.org/jira/browse/HBASE-12663
>             Project: HBase
>          Issue Type: Bug
>          Components: master, security
>    Affects Versions: 1.0.0, 2.0.0
>            Reporter: Matteo Bertozzi
>            Assignee: Matteo Bertozzi
>         Attachments: HBASE-12663-v0.patch
>
>
> currently we have getTableDescriptors()/listTableNames()
> and listTableDescriptorsByNamespace()/listTableNamesByNamespace()
> which are just filtering by namespace.
> The first two are already able to filter tables by regex,
> and there is also an ACL check which shows only the table the user have 
> access to,
> while the namespace version just return a list without any ACLs check.
> to me there is no point to keep the *ByNamespace() version on the server 
> side, 
> since the "base" version must be able to do the filtering by
> namespace and the ACLs check must be the same for both anyway.
> or at least we can just call the base version with the ns filtering.



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

Reply via email to