[
https://issues.apache.org/jira/browse/SENTRY-1536?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15832673#comment-15832673
]
Hadoop QA commented on SENTRY-1536:
-----------------------------------
Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12848662/SENTRY-1536.002-sentry-ha-redesign.patch
against sentry-ha-redesign.
{color:red}Overall:{color} -1 due to 7 errors
{color:red}ERROR:{color} mvn test exited 1
{color:red}ERROR:{color} Failed:
org.apache.sentry.provider.db.tools.TestSentryShellHive
{color:red}ERROR:{color} Failed:
org.apache.sentry.provider.db.tools.TestSentryShellHive
{color:red}ERROR:{color} Failed:
org.apache.sentry.provider.db.tools.TestSentryShellHive
{color:red}ERROR:{color} Failed:
org.apache.sentry.provider.db.tools.TestSentryShellHive
{color:red}ERROR:{color} Failed:
org.apache.sentry.provider.db.tools.TestSentryShellHive
{color:red}ERROR:{color} Failed:
org.apache.sentry.provider.db.tools.TestSentryShellHive
Console output:
https://builds.apache.org/job/PreCommit-SENTRY-Build/2270/console
This message is automatically generated.
> Refactor SentryStore transaction management to allow for extra transactions
> for a single permission update
> ----------------------------------------------------------------------------------------------------------
>
> Key: SENTRY-1536
> URL: https://issues.apache.org/jira/browse/SENTRY-1536
> Project: Sentry
> Issue Type: Sub-task
> Components: Sentry
> Affects Versions: sentry-ha-redesign
> Reporter: Alexander Kolbasov
> Assignee: Hao Hao
> Labels: hdfs-sync
> Fix For: sentry-ha-redesign
>
> Attachments: SENTRY-1536.001-sentry-ha-redesign.patch,
> SENTRY-1536.002-sentry-ha-redesign.patch
>
>
> To persist single permission change, it needs to combine multiple things in a
> single transaction:
> * Doing the actual operation (priv change)
> * Updating notification ID.
> It is important to do this in a single transaction to guarantee that
> notificationID handling is atomic. Current code structure doesn't allow for
> that.
> So we need to pass extra transaction code to SentryStore functions or figure
> out a generic way to do this.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
