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

Sravya Tirukkovalur commented on SENTRY-1390:
---------------------------------------------

Ok, synced up with Hao offline as it is was getting tricky to communicate over 
jira. So here is the summary:
- URI handling, that is handling missing scheme or authority happens in 
implies() in all paths (HMS and HS2), which is fixed in Sentry-1311. 
- This patch is needed as one of the test is failing in the setup without this 
change. But seems like this change does have impact else where. So I think it 
is hard to access the regression risk with out knowing the full impact. 

[~hahao] if you agree  - how about lets
-  Change the description of the jira, as this has nothing to do with HMS 
binding?
- Make this test only change and investigate the setup failure as a follow on 
if needed when we know the full context of the impact?

Thanks,

> Enabling usability of URI privileges for HMS binding
> ----------------------------------------------------
>
>                 Key: SENTRY-1390
>                 URL: https://issues.apache.org/jira/browse/SENTRY-1390
>             Project: Sentry
>          Issue Type: Task
>          Components: URI
>    Affects Versions: 1.8.0
>            Reporter: Hao Hao
>            Assignee: Hao Hao
>         Attachments: SENTRY-1390.0.patch, SENTRY-1390.1.patch, 
> SENTRY-1390.2.patch
>
>
> Enabling usability of URI privileges for HMS write protection, such that URI 
> with no scheme or authority can be handled. e.g 
>   * GRANT ALL ON URI 'hdfs://namenode:XXX/path/to/table'
>   * CREATE EXTERNAL TABLE foo LOCATION '/path/to/table'



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

Reply via email to