KeDeng has posted comments on this change. ( http://gerrit.cloudera.org:8080/19622 )
Change subject: KUDU-3413 [multi-tenancy] add tenant info in metadata for multi-tenancy ...................................................................... Patch Set 16: (3 comments) Thanks for your reviews. http://gerrit.cloudera.org:8080/#/c/19622/15/src/kudu/server/server_base_options.h File src/kudu/server/server_base_options.h: http://gerrit.cloudera.org:8080/#/c/19622/15/src/kudu/server/server_base_options.h@67 PS15, Line 67: > Only used in tests ? Add some comments if it's true. Not like this. Except for unit tests, this structure can be used in the new cluster to initialize the default tenant key, corresponding to the existing server key. http://gerrit.cloudera.org:8080/#/c/19622/15/src/kudu/tools/tool_action_fs.cc File src/kudu/tools/tool_action_fs.cc: http://gerrit.cloudera.org:8080/#/c/19622/15/src/kudu/tools/tool_action_fs.cc@125 PS15, Line 125: LOG(ERROR) << "'tenant_name', 'tenant_id', 'tenant_key', 'tenant_key_iv', and " > nit: add spaces Done http://gerrit.cloudera.org:8080/#/c/19622/15/src/kudu/tools/tool_action_fs.cc@294 PS15, Line 294: !FLAGS_tenant_name.empty()) { : tenant_name = FLAGS_tenant_name; : tenant_id = FLAGS_tenant_id; : encryption_key = FLAGS_tenant_key; : encryption_key_iv = FLAGS_tenant_key_i > They have been validated by GROUP_FLAG_VALIDATOR, here just check FLAGS_ten Done -- To view, visit http://gerrit.cloudera.org:8080/19622 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I9e450d73940eb1dbaac6f905a46d6ccd084f15cf Gerrit-Change-Number: 19622 Gerrit-PatchSet: 16 Gerrit-Owner: KeDeng <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Ashwani Raina <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: KeDeng <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Wang Xixu <[email protected]> Gerrit-Reviewer: Yifan Zhang <[email protected]> Gerrit-Reviewer: Yingchun Lai <[email protected]> Gerrit-Reviewer: Yuqi Du <[email protected]> Gerrit-Comment-Date: Fri, 09 Jun 2023 11:43:42 +0000 Gerrit-HasComments: Yes
