You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sentry.apache.org by "SentryQA (JIRA)" <ji...@apache.org> on 2014/06/27 23:59:24 UTC

[jira] [Commented] (SENTRY-317) Fix TestDbOperations.testLoad test

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

SentryQA commented on SENTRY-317:
---------------------------------

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

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

{color:red}ERROR:{color} mvn test exited 1
{color:red}ERROR:{color} Failed: org.apache.sentry.provider.db.service.persistent.TestSentryStore
{color:red}ERROR:{color} Failed: org.apache.sentry.provider.db.service.persistent.TestSentryStore
{color:red}ERROR:{color} Failed: org.apache.sentry.provider.db.service.persistent.TestSentryStore

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

This message is automatically generated.

> Fix TestDbOperations.testLoad test
> ----------------------------------
>
>                 Key: SENTRY-317
>                 URL: https://issues.apache.org/jira/browse/SENTRY-317
>             Project: Sentry
>          Issue Type: Bug
>    Affects Versions: 1.4.0
>            Reporter: Sravya Tirukkovalur
>            Assignee: Sravya Tirukkovalur
>             Fix For: 1.4.0
>
>         Attachments: SENTRY-317.patch
>
>
> This test is consistently failing. 
> Looks like in policy files we do support uri privileges of the form:
> file:////path (4 slashes)
> but when using db based provider file:///path is not translated to stored file:////path, and hence fails the authorization check.
> Not sure if we want to support 4 slashes format, this patch just fixes the test. Lets file a follow on jira if we want to support 4 slashes.



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