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

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

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

Hadoop QA commented on SENTRY-1510:
-----------------------------------

Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12835191/SENTRY-1510.001-sentry-ha-redesign.patch against sentry-ha-redesign.

{color:red}Overall:{color} -1 due to 2 errors

{color:red}ERROR:{color} mvn test exited 1
{color:red}ERROR:{color} Failed: org.apache.sentry.provider.db.service.thrift.TestSentryServiceFailureCase

Console output: https://builds.apache.org/job/PreCommit-SENTRY-Build/2092/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
>
>
> 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)