Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8878 )
Change subject: KUDU-2228: Make Messenger options configurable ...................................................................... KUDU-2228: Make Messenger options configurable Currently, the RPC layer accesses many gflags directly to take certain decisions, eg. whether to turn on encryption, authentication, etc. Since the RPC layer is to be used more like a library, these should be configurable options that are passed to the Messenger (which is the API endpoint for the application using the RPC layer), instead of the RPC layer itself directly accessing these flags. This patch converts the following flags to Messenger options and moves the flag definitions to server_base.cc which is the "application" in Kudu that uses the Messenger: FLAGS_rpc_default_keepalive_time_ms FLAGS_rpc_negotiation_timeout_ms FLAGS_rpc_authentication FLAGS_rpc_encryption FLAGS_rpc_tls_ciphers FLAGS_rpc_tls_min_protocol FLAGS_rpc_certificate_file FLAGS_rpc_private_key_file FLAGS_rpc_ca_certificate_file FLAGS_rpc_private_key_password_cmd FLAGS_keytab_file Most of the remaining flags are test or benchmark related flags. There may be a few more flags that can be moved out and converted to options, but we can leave that as future work if we decide to move them. In addition to the cherry-pick above, this change also updates Impala code to pass the key_tab file to InitKerberosForServer() which was changed by this Kudu patch. Change-Id: Ia21814ffb6e283c2791985b089878b579905f0ba Reviewed-on: http://gerrit.cloudera.org:8080/8789 Tested-by: Kudu Jenkins Reviewed-by: Dan Burkert <[email protected]> Reviewed-on: http://gerrit.cloudera.org:8080/8878 Reviewed-by: Sailesh Mukil <[email protected]> Tested-by: Impala Public Jenkins --- M be/src/kudu/rpc/client_negotiation.cc M be/src/kudu/rpc/client_negotiation.h M be/src/kudu/rpc/messenger.cc M be/src/kudu/rpc/messenger.h M be/src/kudu/rpc/negotiation.cc M be/src/kudu/rpc/negotiation.h M be/src/kudu/rpc/reactor.cc M be/src/kudu/rpc/rpc-test-base.h M be/src/kudu/rpc/rpc-test.cc M be/src/kudu/rpc/sasl_common.cc M be/src/kudu/rpc/sasl_common.h M be/src/kudu/rpc/server_negotiation.cc M be/src/kudu/rpc/server_negotiation.h M be/src/kudu/security/CMakeLists.txt M be/src/kudu/security/init.cc M be/src/kudu/security/init.h A be/src/kudu/security/security_flags.cc A be/src/kudu/security/security_flags.h M be/src/kudu/security/test/mini_kdc-test.cc M be/src/kudu/security/tls_context.cc M be/src/kudu/security/tls_context.h M be/src/kudu/util/flags.cc M be/src/kudu/util/flags.h M be/src/rpc/authentication.cc 24 files changed, 381 insertions(+), 283 deletions(-) Approvals: Sailesh Mukil: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/8878 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ia21814ffb6e283c2791985b089878b579905f0ba Gerrit-Change-Number: 8878 Gerrit-PatchSet: 4 Gerrit-Owner: Michael Ho <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Lars Volker <[email protected]> Gerrit-Reviewer: Michael Ho <[email protected]> Gerrit-Reviewer: Sailesh Mukil <[email protected]>
