[jira] [Comment Edited] (SENTRY-1510) Add option to use non pool model for sentry client

["Hao Hao (JIRA)" <ji...@apache.org>]

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

Hao Hao edited comment on SENTRY-1510 at 10/26/16 5:26 PM:
-----------------------------------------------------------

+1 Thanks Li!


was (Author: hahao):
+1 Thanks [~liliapache123]!

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