Abhishek Chennaka has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/22415 )

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]>
(cherry picked from commit d405ad33080cc5b203e3789363a202d472aca795)
Reviewed-on: http://gerrit.cloudera.org:8080/22415
Reviewed-by: Ashwani Raina <[email protected]>
Tested-by: Alexey Serbin <[email protected]>
Reviewed-by: Abhishek Chennaka <[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:
  Ashwani Raina: Looks good to me, but someone else must approve
  Alexey Serbin: Verified
  Abhishek Chennaka: Looks good to me, approved

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

Gerrit-Project: kudu
Gerrit-Branch: branch-1.18.x
Gerrit-MessageType: merged
Gerrit-Change-Id: Ic2037582d433730884212e83359bb75bad0d5394
Gerrit-Change-Number: 22415
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[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)

Reply via email to