Alexey Serbin has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/22079 )

Change subject: [compaction] Add memory tracking for better OOM issues triaging
......................................................................

[compaction] Add memory tracking for better OOM issues triaging

This patch adds memory accounting for Major Delta Compaction.
The same memory usage is tracked under newly added MemTracker.
Along with that, throttled warning messages are added when the
memory usage goes beyond a certain threshold.

A unit test is added to verify the presence of warning messages
when memory threshold (reduced for test purpose) is crossed.

Change-Id: Ic2037582d433730884212e83359bb75bad0d5394
Reviewed-on: http://gerrit.cloudera.org:8080/22079
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin <[email protected]>
---
M src/kudu/common/rowblock_memory.h
M src/kudu/tablet/compaction-highmem-test.cc
M src/kudu/tablet/delta_compaction.cc
M src/kudu/tablet/delta_compaction.h
M src/kudu/tablet/delta_store.h
M src/kudu/tablet/deltafile.cc
M src/kudu/util/process_memory.cc
M src/kudu/util/process_memory.h
8 files changed, 141 insertions(+), 7 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Alexey Serbin: Looks good to me, approved

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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ic2037582d433730884212e83359bb75bad0d5394
Gerrit-Change-Number: 22079
Gerrit-PatchSet: 17
Gerrit-Owner: Ashwani Raina <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Ashwani Raina <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy <[email protected]>
Gerrit-Reviewer: Yifan Zhang <[email protected]>

Reply via email to