[jira] [Commented] (SENTRY-2355) Merge the DB owner privileges configurations into one enum configuration

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

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

Hadoop QA commented on SENTRY-2355:
-----------------------------------

Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12935732/SENTRY-2355.1.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.dbprovider.TestOwnerPrivilegesWithGrantOption
{color:red}ERROR:{color} Failed: org.apache.sentry.tests.e2e.dbprovider.TestOwnerPrivilegesWithGrantOption
{color:red}ERROR:{color} Failed: org.apache.sentry.tests.e2e.dbprovider.TestOwnerPrivilegesWithGrantOption
{color:red}ERROR:{color} Failed: org.apache.sentry.tests.e2e.dbprovider.TestOwnerPrivilegesWithGrantOption

Console output: https://builds.apache.org/job/PreCommit-SENTRY-Build/4027/console

This message is automatically generated.

> Merge the DB owner privileges configurations into one enum configuration
> ------------------------------------------------------------------------
>
>                 Key: SENTRY-2355
>                 URL: https://issues.apache.org/jira/browse/SENTRY-2355
>             Project: Sentry
>          Issue Type: Sub-task
>          Components: Sentry
>    Affects Versions: 2.1.0
>            Reporter: Sergio Peña
>            Assignee: Sergio Peña
>            Priority: Major
>         Attachments: SENTRY-2355.1.patch
>
>
> Instead of using 'sentry.enable.owner.privileges' and 'sentry.enable.owner.privileges.with.grant', we could easily merge both configs into one single option that allows users to manage the desired grant for the owner privilege.
> The proposal is to use 'sentry.db.policy.store.owner.as.privilege' which is set to an enum value that contains 'none', 'all' and 'all_with_grant'.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)