[ 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)