[
https://issues.apache.org/jira/browse/SENTRY-662?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15032814#comment-15032814
]
Hadoop QA commented on SENTRY-662:
----------------------------------
Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12701506/SENTRY-662.1.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/1007/console
This message is automatically generated.
> SentryServiceIntegrationBase should use UGI based login
> -------------------------------------------------------
>
> Key: SENTRY-662
> URL: https://issues.apache.org/jira/browse/SENTRY-662
> Project: Sentry
> Issue Type: Improvement
> Affects Versions: 1.5.1
> Reporter: Prasad Mujumdar
> Assignee: Prasad Mujumdar
> Attachments: SENTRY-662.1.patch
>
>
> With Sentry HA the service client makes the service connection on the first
> client api. This is done in order to propagate the exception cleanly to the
> caller.
> The SentryServiceIntegrationBase currently performs direct Jaas login and
> passes the subject to transport open only. Now that any API can open the
> connection, this is not sufficient.
> We should use the UGI login for the tests as well which handles passing the
> right subject to transport level open.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
