[jira] [Commented] (SENTRY-677) Make the Sentry DB provider RPC methods synchronized

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

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

Hadoop QA commented on SENTRY-677:
----------------------------------

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

{color:red}Overall:{color} -1 due to an error

{color:red}ERROR:{color} failed to apply patch (exit code 1):
The patch does not appear to apply with p0, p1, or p2



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

This message is automatically generated.

> Make the Sentry DB provider RPC methods synchronized
> ----------------------------------------------------
>
>                 Key: SENTRY-677
>                 URL: https://issues.apache.org/jira/browse/SENTRY-677
>             Project: Sentry
>          Issue Type: Bug
>    Affects Versions: 1.4.0
>            Reporter: Prasad Mujumdar
>            Assignee: Ryan P
>             Fix For: 1.7.0
>
>         Attachments: SENTRY-677.patch
>
>
> Some of the DB provider RPC handlers are not declared as synchronized. The thrift sync client doesn't handle concurrency very well and hence the handler methods needs to be synchronized. We still have a few like listPrivilegesForProvider() that is not.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)