impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael Ho (Code Review)" <>
Subject [Impala-ASF-CR] KUDU-2228: Make Messenger options configurable
Date Tue, 19 Dec 2017 03:45:47 GMT
Hello Lars Volker, Dan Burkert, Kudu Jenkins, Sailesh Mukil, Joe McDonnell, 

I'd like you to reexamine a change. Please visit

to look at the new patch set (#2).

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 which is the "application" in
Kudu that uses the Messenger:


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.

Change-Id: Ia21814ffb6e283c2791985b089878b579905f0ba
Tested-by: Kudu Jenkins
Reviewed-by: Dan Burkert <>
M be/src/kudu/rpc/
M be/src/kudu/rpc/client_negotiation.h
M be/src/kudu/rpc/
M be/src/kudu/rpc/messenger.h
M be/src/kudu/rpc/
M be/src/kudu/rpc/negotiation.h
M be/src/kudu/rpc/
M be/src/kudu/rpc/rpc-test-base.h
M be/src/kudu/rpc/
M be/src/kudu/rpc/
M be/src/kudu/rpc/sasl_common.h
M be/src/kudu/rpc/
M be/src/kudu/rpc/server_negotiation.h
M be/src/kudu/security/CMakeLists.txt
M be/src/kudu/security/
M be/src/kudu/security/init.h
A be/src/kudu/security/
A be/src/kudu/security/security_flags.h
M be/src/kudu/security/test/
M be/src/kudu/security/
M be/src/kudu/security/tls_context.h
M be/src/kudu/util/
M be/src/kudu/util/flags.h
M be/src/rpc/
24 files changed, 381 insertions(+), 282 deletions(-)

  git pull ssh:// refs/changes/78/8878/2
To view, visit
To unsubscribe, visit

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ia21814ffb6e283c2791985b089878b579905f0ba
Gerrit-Change-Number: 8878
Gerrit-PatchSet: 2
Gerrit-Owner: Michael Ho <>
Gerrit-Reviewer: Dan Burkert <>
Gerrit-Reviewer: Joe McDonnell <>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Lars Volker <>
Gerrit-Reviewer: Michael Ho <>
Gerrit-Reviewer: Sailesh Mukil <>

  • Unnamed multipart/alternative (inline, 8-Bit, 0 bytes)
View raw message