You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Attila Bukor (Code Review)" <ge...@cloudera.org> on 2023/01/27 15:35:59 UTC

[kudu-CR] KUDU-3423 Add support for Ranger KMS HA

Hello Zoltan Chovan, Alexey Serbin, Kudu Jenkins, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/19271

to look at the new patch set (#6).

Change subject: KUDU-3423 Add support for Ranger KMS HA
......................................................................

KUDU-3423 Add support for Ranger KMS HA

Ranger KMS supports high availability by listing multiple Ranger KMS
servers (comma-separated, no spaces). Up until now, Kudu allowed
configuring only one Ranger KMS server.

This patch adds support for listing multiple KMS servers. This is done
by extending EasyCurl to support multiple URLs. Kudu always attempts to
connect to the first one in the list, and if the connection fails due to
a network error or a timeout, it fails over to the next one. If a server
responds with an error, that is considered a valid response and then it
is returned immediately to the caller without trying the remaining URLs.

Change-Id: Ibef941ed20eda1f4e624c2c7e16ca7955af570d8
---
M src/kudu/fs/fs_manager.cc
M src/kudu/integration-tests/security-itest.cc
M src/kudu/ranger-kms/ranger_kms_client.cc
M src/kudu/ranger-kms/ranger_kms_client.h
M src/kudu/util/curl_util.cc
M src/kudu/util/curl_util.h
6 files changed, 104 insertions(+), 12 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/71/19271/6
-- 
To view, visit http://gerrit.cloudera.org:8080/19271
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ibef941ed20eda1f4e624c2c7e16ca7955af570d8
Gerrit-Change-Number: 19271
Gerrit-PatchSet: 6
Gerrit-Owner: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Chovan <zc...@cloudera.com>