[
https://issues.apache.org/jira/browse/SENTRY-755?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14633123#comment-14633123
]
Lenni Kuff commented on SENTRY-755:
-----------------------------------
[~sravya] - Thanks for this change. A few comments:
* Can you ensure the following works (and has test coverage): user has grant on
database/table/server AND some column level privs are present. User should have
HDFS privs because parent grant take precedence.
* Add test case where table contains multiple columns (and consider naming
columns "col1", "col2" for clarity) and user has access to some (but not all)
of the columns (should not have HDFS access). Once user has been granted access
to all columns they should have HDFS privs.
* Add some comments in the test case to clarify what scenarios you are
validating.
* Add a comment someplace in the code to describe the behavior of column-level
privs.
* There's some commented out code in the test
> HDFS access of data files should be disabled for user with privileges only on
> some columns
> ------------------------------------------------------------------------------------------
>
> Key: SENTRY-755
> URL: https://issues.apache.org/jira/browse/SENTRY-755
> Project: Sentry
> Issue Type: New Feature
> Components: Hdfs Plugin, Hive Plugin
> Affects Versions: 1.5.0
> Reporter: Sravya Tirukkovalur
> Assignee: Sravya Tirukkovalur
> Fix For: 1.6.0
>
> Attachments: SENTRY-755-test.patch, SENTRY-755.2.patch
>
>
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
