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 <aser...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: David Ribeiro Alves <dral...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>

Reply via email to