[jira] [Commented] (SENTRY-166) Sentry does not accept URIs with an equals sign (=) in path. Fails with llegalArgumentException: Invalid key value

["SentryQA (JIRA)" <ji...@apache.org>]

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

SentryQA commented on SENTRY-166:
---------------------------------

Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12648890/SENTRY-166.patch against master.

{color:red}Overall:{color} -1 due to 2 errors

{color:red}ERROR:{color} mvn test exited 1
{color:red}ERROR:{color} Failed: org.apache.sentry.provider.file.TestKeyValue

Console output: http://bigtop01.cloudera.org:8080/job/PreCommit-SENTRY-Build/112/console

This message is automatically generated.

> Sentry does not accept URIs with an equals sign (=) in path. Fails with llegalArgumentException: Invalid key value
> ------------------------------------------------------------------------------------------------------------------
>
>                 Key: SENTRY-166
>                 URL: https://issues.apache.org/jira/browse/SENTRY-166
>             Project: Sentry
>          Issue Type: Bug
>    Affects Versions: 1.2.0
>            Reporter: Lenni Kuff
>            Assignee: Jarek Jarcec Cecho
>            Priority: Critical
>              Labels: grant/revoke
>             Fix For: 1.4.0
>
>         Attachments: SENTRY-166.patch
>
>
> Sentry does not accept URIs with an equals sign (=) in path. This is quite important for Impala/Hive because in many cases a partition location will have an equals sign in the path.
> {code}
> java.lang.IllegalArgumentException: Invalid key value: URI=hdfs://c2102.hal.cloudera.com:8020/test-warehouse/a=1 [URI, hdfs://c2102.hal.cloudera.com:8020/test-warehouse/a, 1]
>         at org.apache.sentry.provider.file.KeyValue.<init>(KeyValue.java:33)
>         at org.apache.sentry.policy.db.DBWildcardPermission.<init>(DBWildcardPermission.java:67)
>         at org.apache.sentry.policy.db.DBWildcardPermission$DBWildcardPermissionFactory.createPermission(DBWildcardPermission.java:178)
>         at org.apache.sentry.provider.file.ResourceAuthorizationProvider.doHasAccess(ResourceAuthorizationProvider.java:106)
>         at org.apache.sentry.provider.file.ResourceAuthorizationProvider.hasAccess(ResourceAuthorizationProvider.java:87)
> {code}
> Repro:
> {code}
> create table sentry_test(i int) location '/test-warehouse/a=1';
> {code}



--
This message was sent by Atlassian JIRA
(v6.2#6252)