hadoop-common-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Arun Suresh (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (HADOOP-11620) Add Support for Load Balancing across a group of KMS servers for HA
Date Wed, 25 Feb 2015 06:17:05 GMT

     [ https://issues.apache.org/jira/browse/HADOOP-11620?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel

Arun Suresh updated HADOOP-11620:
    Attachment: HADOOP-11620.5.patch

Thanks for the review [~andrew.wang]

Uploading updated patch with your suggestions.

bq. ..enough to safely increment currentIdx. I guess it's not a big deal, but it'd be safer
to use an AtomicInt here.
Actually atomic wont work here.. since I have to do modulo increment. but ive changed it increment
in a synchronized scope

bq.   ..the createProvider refactor related to this patch? Doesn't seem necessary.
Agreed.. I was thinking of using it something else first.. but I decided to keep it since
copy pasting the constructor for LBKMSClientProvider everywhere looked ugly 

> Add Support for Load Balancing across a group of KMS servers for HA
> -------------------------------------------------------------------
>                 Key: HADOOP-11620
>                 URL: https://issues.apache.org/jira/browse/HADOOP-11620
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: kms
>    Affects Versions: 2.6.0
>            Reporter: Arun Suresh
>            Assignee: Arun Suresh
>         Attachments: HADOOP-11620.1.patch, HADOOP-11620.2.patch, HADOOP-11620.3.patch,
HADOOP-11620.4.patch, HADOOP-11620.5.patch
> This patch needs to add support for :
> * specification of multiple hostnames in the kms key provider uri
> * KMS client to load balance requests across the hosts specified in the kms keyprovider

This message was sent by Atlassian JIRA

View raw message