Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15030 )
Change subject: [tablet] KUDU-3023 validate RPC vs transaction size limit ...................................................................... [tablet] KUDU-3023 validate RPC vs transaction size limit Added a group validator to enforce consistency between the maximum RPC size and the per-tablet transaction memory limit. This is to make it possible to apply big enough write requests replicated by Raft (i.e., very close to the maximum allowed RPC size) by the local replica itself. I also added a validator for the --tablet_transaction_memory_limit_mb flag and updated a test that was using custom setting for the flag to pass the newly added group flag validator. Change-Id: Ia9384038a078027e997d7774b2be0e753c822fec Reviewed-on: http://gerrit.cloudera.org:8080/15030 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/integration-tests/raft_consensus-itest.cc M src/kudu/tablet/transactions/transaction_tracker.cc 2 files changed, 37 insertions(+), 5 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/15030 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ia9384038a078027e997d7774b2be0e753c822fec Gerrit-Change-Number: 15030 Gerrit-PatchSet: 3 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
