Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3668
to look at the new patch set (#4).
Change subject: KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode
......................................................................
KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode
Implemented AUTO_FLUSH_BACKGROUND for C++ client library.
KuduSession instance starts a dedicated background flusher thread
to perform flush in background. In AUTO_FLUSH_BACKGROUND mode,
the KuduSession::Apply() method blocks if running out of space
in 'virtual buffer': there isn't a single buffer as is,
there is a set of buffered write operations used by multiple batchers.
The limit on the buffer size can be set by
KuduSession::SetMutationBufferSpace().
The background flusher uses high-watermark approach for
its flush criterion: it counts buffer size used by not-yet-flushed
write operations.
The data flow control implemented by KuduSession::Apply() combines
high-/low-watermark approach for its pause/continue criterion:
it counts total buffer size used by write operations.
This change also addresses the following JIRA issue(s):
KUDU-1376 KuduSession::SetMutationBufferSpace is not defined
Change-Id: Idfde05bef342db24990c6e3da3b0270c3bb37a9d
---
M src/kudu/client/batcher.cc
M src/kudu/client/batcher.h
M src/kudu/client/client-test.cc
M src/kudu/client/client.cc
M src/kudu/client/client.h
M src/kudu/client/session-internal.cc
M src/kudu/client/session-internal.h
M src/kudu/util/monotime.cc
M src/kudu/util/monotime.h
9 files changed, 869 insertions(+), 114 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/68/3668/4
--
To view, visit http://gerrit.cloudera.org:8080/3668
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Idfde05bef342db24990c6e3da3b0270c3bb37a9d
Gerrit-PatchSet: 4
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: David Ribeiro Alves <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
