[jira] [Commented] (SENTRY-2539) PolicyEngine should be able to return privilege directly
[ https://issues.apache.org/jira/browse/SENTRY-2539?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16997871#comment-16997871 ] Hadoop QA commented on SENTRY-2539: --- Here are the results of testing the latest attachment https://issues.apache.org/jira/secure/attachment/12988958/SENTRY-2539.003.patch against master. {color:red}Overall:{color} -1 due to 5 errors {color:red}ERROR:{color} mvn test exited 1 {color:red}ERROR:{color} Failed: org.apache.sentry.tests.e2e.metastore.TestMetastoreEndToEnd {color:red}ERROR:{color} Failed: org.apache.sentry.tests.e2e.metastore.TestMetastoreEndToEnd {color:red}ERROR:{color} Failed: org.apache.sentry.tests.e2e.dbprovider.TestDbSandboxOps {color:red}ERROR:{color} Failed: org.apache.sentry.tests.e2e.dbprovider.TestDbSandboxOps Console output: https://builds.apache.org/job/PreCommit-SENTRY-Build/4456/console This message is automatically generated. > PolicyEngine should be able to return privilege directly > - > > Key: SENTRY-2539 > URL: https://issues.apache.org/jira/browse/SENTRY-2539 > Project: Sentry > Issue Type: Improvement > Components: Sentry >Affects Versions: 2.1 >Reporter: Na Li >Assignee: Na Li >Priority: Major > Attachments: SENTRY-2539.002.patch, SENTRY-2539.003.patch, > SENTRY-2539.005.patch > > > Right now, the PolicyEngine Interface only returns the list of privileges in > the form of String. As a result, every authorization check has to convert the > privilege string to privilege object even though the cached privilege objects > are of the correct type already. > We should add a new function that returns privilege object directly to avoid > the overhead of conversion. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Updated] (SENTRY-2539) PolicyEngine should be able to return privilege directly
[ https://issues.apache.org/jira/browse/SENTRY-2539?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Na Li updated SENTRY-2539: -- Attachment: SENTRY-2539.005.patch > PolicyEngine should be able to return privilege directly > - > > Key: SENTRY-2539 > URL: https://issues.apache.org/jira/browse/SENTRY-2539 > Project: Sentry > Issue Type: Improvement > Components: Sentry >Affects Versions: 2.1 >Reporter: Na Li >Assignee: Na Li >Priority: Major > Attachments: SENTRY-2539.002.patch, SENTRY-2539.003.patch, > SENTRY-2539.005.patch > > > Right now, the PolicyEngine Interface only returns the list of privileges in > the form of String. As a result, every authorization check has to convert the > privilege string to privilege object even though the cached privilege objects > are of the correct type already. > We should add a new function that returns privilege object directly to avoid > the overhead of conversion. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Updated] (SENTRY-2539) PolicyEngine should be able to return privilege directly
[ https://issues.apache.org/jira/browse/SENTRY-2539?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Na Li updated SENTRY-2539: -- Attachment: SENTRY-2539.003.patch > PolicyEngine should be able to return privilege directly > - > > Key: SENTRY-2539 > URL: https://issues.apache.org/jira/browse/SENTRY-2539 > Project: Sentry > Issue Type: Improvement > Components: Sentry >Affects Versions: 2.1 >Reporter: Na Li >Assignee: Na Li >Priority: Major > Attachments: SENTRY-2539.002.patch, SENTRY-2539.003.patch > > > Right now, the PolicyEngine Interface only returns the list of privileges in > the form of String. As a result, every authorization check has to convert the > privilege string to privilege object even though the cached privilege objects > are of the correct type already. > We should add a new function that returns privilege object directly to avoid > the overhead of conversion. -- This message was sent by Atlassian Jira (v8.3.4#803005)