[
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 uri.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
