Todd Lipcon has submitted this change and it was merged.

Change subject: Memory tracking for result tracker
......................................................................


Memory tracking for result tracker

This adds memory tracking to ResultTracker, making sure we account for
the memory as we cache responses for clients' requests.

Testing wise this adds memory consumption checks to rpc-stress-test.cc.

Original patch by David Alves.

Change-Id: I3b81dda41c8bc7f70380ce426142c34afe6f1625
Reviewed-on: http://gerrit.cloudera.org:8080/3627
Tested-by: Kudu Jenkins
Reviewed-by: Adar Dembo <a...@cloudera.com>
---
M src/kudu/rpc/result_tracker.cc
M src/kudu/rpc/result_tracker.h
M src/kudu/rpc/rpc-stress-test.cc
M src/kudu/rpc/rpc-test-base.h
M src/kudu/server/server_base.cc
M src/kudu/util/test_util.cc
M src/kudu/util/test_util.h
7 files changed, 234 insertions(+), 17 deletions(-)

Approvals:
  Adar Dembo: Looks good to me, approved
  Kudu Jenkins: Verified



-- 
To view, visit http://gerrit.cloudera.org:8080/3627
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I3b81dda41c8bc7f70380ce426142c34afe6f1625
Gerrit-PatchSet: 22
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: David Ribeiro Alves <dral...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: David Ribeiro Alves <dral...@apache.org>
Gerrit-Reviewer: Jean-Daniel Cryans <jdcry...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>

Reply via email to