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 (#14).
Change subject: KUDU-3413 [multi-tenancy] add tenant info in metadata for
multi-tenancy
......................................................................
KUDU-3413 [multi-tenancy] 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, 769 insertions(+), 93 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/22/19622/14
--
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: 14
Gerrit-Owner: KeDeng <kdeng...@gmail.com>
Gerrit-Reviewer: Alexey Serbin <ale...@apache.org>
Gerrit-Reviewer: Ashwani Raina <ara...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <abu...@apache.org>
Gerrit-Reviewer: KeDeng <kdeng...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: Wang Xixu <1450306...@qq.com>
Gerrit-Reviewer: Yifan Zhang <chinazhangyi...@163.com>
Gerrit-Reviewer: Yingchun Lai <laiyingc...@apache.org>
Gerrit-Reviewer: Yuqi Du <shenxingwuy...@gmail.com>
