Hello Tidy Bot, Alexey Serbin, Yuqi Du, Ashwani Raina, Yingchun Lai, Attila
Bukor, Yifan Zhang, Kudu Jenkins, Wang Xixu,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/19622
to look at the new patch set (#17).
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
---
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(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/22/19622/17
--
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: newpatchset
Gerrit-Change-Id: I9e450d73940eb1dbaac6f905a46d6ccd084f15cf
Gerrit-Change-Number: 19622
Gerrit-PatchSet: 17
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]>
