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

Hadoop QA commented on SENTRY-662:
----------------------------------

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

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

{color:red}ERROR:{color} mvn test exited 1

Console output: https://builds.apache.org/job/PreCommit-SENTRY-Build/474/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: Bug
>            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)

Reply via email to