[
https://issues.apache.org/jira/browse/SENTRY-1510?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15607758#comment-15607758
]
Hadoop QA commented on SENTRY-1510:
-----------------------------------
Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12835272/SENTRY-1510.003-sentry-ha-redesign.patch
against sentry-ha-redesign.
{color:green}Overall:{color} +1 all checks pass
{color:green}SUCCESS:{color} all tests passed
Console output:
https://builds.apache.org/job/PreCommit-SENTRY-Build/2096/console
This message is automatically generated.
> Add option to use non pool model for sentry client
> --------------------------------------------------
>
> Key: SENTRY-1510
> URL: https://issues.apache.org/jira/browse/SENTRY-1510
> Project: Sentry
> Issue Type: Sub-task
> Affects Versions: sentry-ha-redesign
> Reporter: Li Li
> Assignee: Li Li
> Fix For: sentry-ha-redesign
>
> Attachments: SENTRY-1510.001-sentry-ha-redesign.patch,
> SENTRY-1510.002-sentry-ha-redesign.patch,
> SENTRY-1510.003-sentry-ha-redesign.patch
>
>
> Currently sentry clients e.g. impala uses non pool model
> (SentryPolicyServiceClientDefaultImpl), thus it's better to keep the non pool
> model for those clients to avoid unnecessary incompatible issues.
> Also the current pool model (PoolClientInvocationHandler) does not manage the
> opening connections well. e.g. Opening connections with failed servers should
> be closed promptly.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)