Hello David Ribeiro Alves, Adar Dembo,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/6611
to review the following change.
Change subject: WIP: release WritableLogSegment buffers when log is idle
......................................................................
WIP: release WritableLogSegment buffers when log is idle
This makes the log appender thread wake up after 1 second of inactivity
and release any memory held in the compression buffer of the writable
log segment.
This is based on an observation on a TS with a lot of tablets that a
good amount of memory is held in these buffers even when there is no
active write workload. It would be good to free them up when they are
providing no value.
WIP because I'd like to test this against a real set of data dirs to
compare memory usage.
Change-Id: I512c7f9264ac9490e6a57259e4d7b2608adc1ca7
---
M src/kudu/consensus/log.cc
M src/kudu/consensus/log.h
M src/kudu/consensus/log_util.h
M src/kudu/util/blocking_queue-test.cc
M src/kudu/util/blocking_queue.h
M src/kudu/util/faststring.cc
M src/kudu/util/faststring.h
7 files changed, 95 insertions(+), 19 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/11/6611/1
--
To view, visit http://gerrit.cloudera.org:8080/6611
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I512c7f9264ac9490e6a57259e4d7b2608adc1ca7
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: David Ribeiro Alves <[email protected]>
