[ 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)