Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/20528
to look at the new patch set (#2).
Change subject: WIP [rpc] Set rpc_max_message_size per available memory to
accomodate huge response payloads
......................................................................
WIP [rpc] Set rpc_max_message_size per available memory to accomodate huge
response payloads
It has been observed that for certain Kudu CLI commands response payload
size is too big to fit in the current default rpc message size limit of 50MiB.
This patch add logic to set the value of rpc message max size based on maximum
available memory or maximum possible rpc message size limit of 2GiB. This would
help accomodate that extra heavy response payload.
Along with increasing default value of rpc_max_message_size, default value
of tablet_transaction_memory_limit_mb is set to autosize i.e. 0. This ensures
the limit is chosen based on what rpc_max_message_size is at the time.
Change-Id: I9c92e5469f806c827a8353fdf6de5a24a221613c
---
M src/kudu/consensus/consensus_queue.cc
M src/kudu/integration-tests/master_replication-itest.cc
M src/kudu/integration-tests/raft_consensus-itest.cc
M src/kudu/master/catalog_manager.cc
M src/kudu/master/sys_catalog.cc
M src/kudu/rpc/blocking_ops.cc
M src/kudu/rpc/rpc-test-base.h
M src/kudu/rpc/rpc-test.cc
M src/kudu/rpc/serialization.cc
M src/kudu/rpc/transfer.cc
M src/kudu/rpc/transfer.h
M src/kudu/tablet/ops/op_tracker-test.cc
M src/kudu/tablet/ops/op_tracker.cc
M src/kudu/tools/kudu-tool-test.cc
M src/kudu/tserver/tablet_copy_service.cc
M src/kudu/tserver/tablet_copy_source_session.cc
16 files changed, 96 insertions(+), 43 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/28/20528/2
--
To view, visit http://gerrit.cloudera.org:8080/20528
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I9c92e5469f806c827a8353fdf6de5a24a221613c
Gerrit-Change-Number: 20528
Gerrit-PatchSet: 2
Gerrit-Owner: Ashwani Raina <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
