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

Reply via email to