[jira] [Comment Edited] (SENTRY-1593) Implement client failover for Generic and NN clients

["kalyan kumar kalvagadda (JIRA)" <ji...@apache.org>]

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

kalyan kumar kalvagadda edited comment on SENTRY-1593 at 4/4/17 5:10 PM:
-------------------------------------------------------------------------


At a high level the change set submitted does following.
1. Extends the capability of sentry name-node client and generic policy client to connect to multiple servers and failover to the server that is available
2. Transport handling of the clients is abstracted to another class so that the client's just have to extend the class that abstracts it. Implementations of the clients don't have to worry about it.
3. Changed the behavior of the clients so that they connect to one of the configured servers when the client is actually instantiated.

As part of the code changes there are couple of files that are moved from one package to another. Here is the list of them
# RetryClientInvocationHandler.java
# SentryClientInvocationHandler.java
# ThriftUtil.java
# SentryHdfsServiceException.java


Amount these files,  RetryClientInvocationHandler.java has changed. Please have a closer look at this file.


was (Author: kkalyan):
As part of the code changes there are couple of files that are moved from one package to another. Here is the list of them
# RetryClientInvocationHandler.java
# SentryClientInvocationHandler.java
# ThriftUtil.java
# SentryHdfsServiceException.java


Amount these files,  RetryClientInvocationHandler.java has changed. Please have a closer look at this file.

> Implement client failover for Generic and NN clients
> ----------------------------------------------------
>
>                 Key: SENTRY-1593
>                 URL: https://issues.apache.org/jira/browse/SENTRY-1593
>             Project: Sentry
>          Issue Type: Sub-task
>          Components: Sentry
>    Affects Versions: sentry-ha-redesign
>            Reporter: Alexander Kolbasov
>            Assignee: kalyan kumar kalvagadda
>            Priority: Blocker
>              Labels: HA
>             Fix For: sentry-ha-redesign
>
>         Attachments: old_service_client_class_diagram.png, SENTRY-1593.001-sentry-ha-redesign.patch, SENTRY-1593.002-sentry-ha-redesign.patch, SENTRY-1593.003-sentry-ha-redesign.patch, SENTRY-1593.004-sentry-ha-redesign.patch, SENTRY-1593.005-sentry-ha-redesign.patch, SENTRY-1593.006-sentry-ha-redesign.patch, service_client_class_diagram.png
>
>
> We need to have client failover logic for Generic service clients and Name Node clients. Currently only db policy clients have it implemented.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)