KeDeng has posted comments on this change. ( http://gerrit.cloudera.org:8080/19622 )
Change subject: KUDU-3413 [multi-tenancy] update server key for multi-tenancy ...................................................................... Patch Set 10: (2 comments) http://gerrit.cloudera.org:8080/#/c/19622/9/src/kudu/util/env_posix.cc File src/kudu/util/env_posix.cc: http://gerrit.cloudera.org:8080/#/c/19622/9/src/kudu/util/env_posix.cc@210 PS9, Line 210: DEFINE_validator(encryption_key_length, > Does it matter? You can DECLARE_bool(enable_multi_tenancy) anywhere after d Yes, it can usually be used this way. However, if there are scenarios involving linked libraries, there may be some discrepancies. If A needs to link B, then B should not be able to use the gflag defined in A. Now fs_manager.cc file needs to be linked to kudu_util library, so the best way is to define this flag in kudu_util for upper level use. http://gerrit.cloudera.org:8080/#/c/19622/10/src/kudu/util/test_util.cc File src/kudu/util/test_util.cc: http://gerrit.cloudera.org:8080/#/c/19622/10/src/kudu/util/test_util.cc@93 PS10, Line 93: static const char* const kEncryptionTenantName = "default_tenant_kudu"; : static const char* const kEncryptionTenantID = "00000000000000000000000000000000"; > Can we reuse the variables in https://gerrit.cloudera.org/c/19622/10/src/ku These two variables actually have different usage scenarios. The name used for this test can be different from what we actually use, and future implementations will add this scenario, so I think it can be kept separate. -- 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: 10 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: Tue, 16 May 2023 02:16:42 +0000 Gerrit-HasComments: Yes
