Hello Kudu Jenkins,

I'd like you to reexamine a change.  Please visit

    http://gerrit.cloudera.org:8080/6611

to look at the new patch set (#2).

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/consensus_peers-test.cc
M src/kudu/consensus/consensus_queue-test.cc
M src/kudu/consensus/log-test-base.h
M src/kudu/consensus/log.cc
M src/kudu/consensus/log.h
M src/kudu/consensus/log_cache-test.cc
M src/kudu/consensus/log_util.h
M src/kudu/consensus/raft_consensus_quorum-test.cc
M src/kudu/integration-tests/ts_recovery-itest.cc
M src/kudu/tablet/tablet_bootstrap.cc
M src/kudu/util/CMakeLists.txt
M src/kudu/util/blocking_queue-test.cc
M src/kudu/util/blocking_queue.h
A src/kudu/util/faststring-test.cc
M src/kudu/util/faststring.cc
M src/kudu/util/faststring.h
16 files changed, 186 insertions(+), 32 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/11/6611/2
-- 
To view, visit http://gerrit.cloudera.org:8080/6611
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I512c7f9264ac9490e6a57259e4d7b2608adc1ca7
Gerrit-PatchSet: 2
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]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot

Reply via email to