Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/19271 )
Change subject: KUDU-3423 Add support for Ranger KMS HA ...................................................................... Patch Set 3: (4 comments) http://gerrit.cloudera.org:8080/#/c/19271/2//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/19271/2//COMMIT_MSG@10 PS2, Line 10: server > nit: servers Done http://gerrit.cloudera.org:8080/#/c/19271/2//COMMIT_MSG@16 PS2, Line 16: If a server : responds with an error, that > I think this needs to be cleared up a bit. Are you referring to the first s Added a note about it. http://gerrit.cloudera.org:8080/#/c/19271/2/src/kudu/fs/fs_manager.cc File src/kudu/fs/fs_manager.cc: http://gerrit.cloudera.org:8080/#/c/19271/2/src/kudu/fs/fs_manager.cc@140 PS2, Line 140: URLs. > nit: URLs Done http://gerrit.cloudera.org:8080/#/c/19271/2/src/kudu/util/curl_util.h File src/kudu/util/curl_util.h: http://gerrit.cloudera.org:8080/#/c/19271/2/src/kudu/util/curl_util.h@59 PS2, Line 59: // Same as above, but with failover support. If the connection to the first > maybe add a note to the comments for the multiple url variants, here and be Done -- 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: comment Gerrit-Change-Id: Ibef941ed20eda1f4e624c2c7e16ca7955af570d8 Gerrit-Change-Number: 19271 Gerrit-PatchSet: 3 Gerrit-Owner: Attila Bukor <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Chovan <[email protected]> Gerrit-Comment-Date: Wed, 23 Nov 2022 16:16:22 +0000 Gerrit-HasComments: Yes
