Yingchun Lai has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/19622 )
Change subject: KUDU-3413 add tenant info in metadata for multi-tenancy ...................................................................... KUDU-3413 add tenant info in metadata for multi-tenancy If we want to introduce multi-tenancy feature, it is important to add relevant metadata information first. In this patch, I added definition of tenant information in metadata, and refactored some metadata loading to distinguish between data at rest encryption feature and multi-tenancy feature. In the subsequent use, if we encounter a situation where both server key and tenant key are present in the metadata, we need to pay attention to the fact that tenant key will have a higher priority. That is, we will consider it as enabling the multi-tenancy feature and ignore the processing of server key. Of course, it is also necessary to provide a way to upgrade server key to tenant key, which I plan to implement in the next patch. In this commit, I added a new flag '--enable_multi_tenancy' to indicate whether we enable multi-tenancy feature. However, considering that the development of the entire feature has not been completed, it is not recommended to set this flag temporarily. We need to wait until the entire feature is developed before considering whether to set it. And I also added unit tests for the new open process to ensure the open logic is effective. Change-Id: I9e450d73940eb1dbaac6f905a46d6ccd084f15cf Reviewed-on: http://gerrit.cloudera.org:8080/19622 Tested-by: Kudu Jenkins Reviewed-by: Yingchun Lai <[email protected]> --- M src/kudu/consensus/raft_consensus_quorum-test.cc M src/kudu/fs/dir_manager.h M src/kudu/fs/fs.proto M src/kudu/fs/fs_manager-test.cc M src/kudu/fs/fs_manager.cc M src/kudu/fs/fs_manager.h M src/kudu/integration-tests/raft_consensus-itest.cc M src/kudu/mini-cluster/external_mini_cluster.cc M src/kudu/mini-cluster/external_mini_cluster.h M src/kudu/mini-cluster/internal_mini_cluster.cc M src/kudu/ranger-kms/ranger_kms_client.cc M src/kudu/server/server_base.cc M src/kudu/server/server_base_options.cc M src/kudu/server/server_base_options.h M src/kudu/tools/kudu-tool-test.cc M src/kudu/tools/tool_action_common.cc M src/kudu/tools/tool_action_fs.cc M src/kudu/tserver/tablet_copy_client-test.cc M src/kudu/tserver/tablet_server-test-base.cc M src/kudu/util/env-test.cc M src/kudu/util/env_posix.cc M src/kudu/util/flag_validators-test.cc M src/kudu/util/test_util.cc M src/kudu/util/test_util.h 24 files changed, 765 insertions(+), 93 deletions(-) Approvals: Kudu Jenkins: Verified Yingchun Lai: Looks good to me, approved -- 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: merged Gerrit-Change-Id: I9e450d73940eb1dbaac6f905a46d6ccd084f15cf Gerrit-Change-Number: 19622 Gerrit-PatchSet: 18 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]>
