KeDeng has posted comments on this change. ( http://gerrit.cloudera.org:8080/19899 )
Change subject: KUDU-3413 [multi-tenancy] add some renames for multi-tenancy ...................................................................... Patch Set 5: (5 comments) Thanks for your reviews. http://gerrit.cloudera.org:8080/#/c/19899/4/src/kudu/fs/fs_manager.cc File src/kudu/fs/fs_manager.cc: http://gerrit.cloudera.org:8080/#/c/19899/4/src/kudu/fs/fs_manager.cc@508 PS4, Line 508: decrypted_key; > It should be a 'decrypted key' because it is decrypted from an encrypted ke Done http://gerrit.cloudera.org:8080/#/c/19899/4/src/kudu/fs/key_provider.h File src/kudu/fs/key_provider.h: http://gerrit.cloudera.org:8080/#/c/19899/4/src/kudu/fs/key_provider.h@38 PS4, Line 38: key. > nit: duplicatd? Done http://gerrit.cloudera.org:8080/#/c/19899/4/src/kudu/fs/ranger_kms_key_provider.h File src/kudu/fs/ranger_kms_key_provider.h: http://gerrit.cloudera.org:8080/#/c/19899/4/src/kudu/fs/ranger_kms_key_provider.h@40 PS4, Line 40: decrypted_key) > Corresponding to https://gerrit.cloudera.org/c/19899/4/src/kudu/fs/key_prov Done http://gerrit.cloudera.org:8080/#/c/19899/4/src/kudu/fs/ranger_kms_key_provider.cc File src/kudu/fs/ranger_kms_key_provider.cc: http://gerrit.cloudera.org:8080/#/c/19899/4/src/kudu/fs/ranger_kms_key_provider.cc@30 PS4, Line 30: decrypted_key) > Corresponding to https://gerrit.cloudera.org/c/19899/4/src/kudu/fs/key_prov Done http://gerrit.cloudera.org:8080/#/c/19899/4/src/kudu/tools/tool_action_common.cc File src/kudu/tools/tool_action_common.cc: http://gerrit.cloudera.org:8080/#/c/19899/4/src/kudu/tools/tool_action_common.cc@975 PS4, Line 975: decrypted_key; > nit: decrypted_key ? Done -- To view, visit http://gerrit.cloudera.org:8080/19899 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ic5bf51df9e7b9ebdaeb353049274cb18fa73a43a Gerrit-Change-Number: 19899 Gerrit-PatchSet: 5 Gerrit-Owner: KeDeng <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: KeDeng <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Yingchun Lai <[email protected]> Gerrit-Reviewer: Yuqi Du <[email protected]> Gerrit-Comment-Date: Fri, 19 May 2023 02:35:23 +0000 Gerrit-HasComments: Yes
