Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9164 )
Change subject: IMPALA-6448: Re-enable kerberized testing with KRPC ...................................................................... IMPALA-6448: Re-enable kerberized testing with KRPC For the patch for IMPALA-5054, we realized that we needed to make the kudu::rpc::Messenger configurable. A patch was done on the Kudu side which is tracked by KUDU-2228. As part of that patch, one of the design decisions taken was to only allow kerberos either on or off for the entirety of the process life. This means that we cannot switch kerberos on and off in the same process any more with KRPC. This behavior can be found in SaslInit() in kudu/rpc/sasl_common.cc as SaslInit() which is called once per process will hard code some configuration which cannot be toggled. This affected our kerberized rpc-mgr-tests. This patch splits out the kerberized part of rpc-mgr-test into rpc-mgr-kerberized-test. It also puts the common code between both the files into rpc-mgr-test-base.h Change-Id: I6412978316de90875c98f8fbe51c8d215c227b18 Reviewed-on: http://gerrit.cloudera.org:8080/9164 Reviewed-by: Sailesh Mukil <sail...@cloudera.com> Tested-by: Impala Public Jenkins --- M be/src/rpc/CMakeLists.txt A be/src/rpc/rpc-mgr-kerberized-test.cc A be/src/rpc/rpc-mgr-test-base.h M be/src/rpc/rpc-mgr-test.cc 4 files changed, 380 insertions(+), 316 deletions(-) Approvals: Sailesh Mukil: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/9164 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I6412978316de90875c98f8fbe51c8d215c227b18 Gerrit-Change-Number: 9164 Gerrit-PatchSet: 5 Gerrit-Owner: Sailesh Mukil <sail...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Michael Ho <k...@cloudera.com> Gerrit-Reviewer: Sailesh Mukil <sail...@cloudera.com>
