[
https://issues.apache.org/jira/browse/SENTRY-662?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14341207#comment-14341207
]
Prasad Mujumdar commented on SENTRY-662:
----------------------------------------
[~dapengsun] Thanks for taking a look. A review request is created for the
patch.
TestHDFSIntegration passes locally for me. I guess we still have some issues
with test. This patch is only changes a few unrelated tests and shouldn't cause
the TestHDFSIntegration failure.
> 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)
