Hello Dan Burkert, Todd Lipcon,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/5533
to review the following change.
Change subject: KUDU-1767. c++ client: Prevent concurrent flushes by default
......................................................................
KUDU-1767. c++ client: Prevent concurrent flushes by default
This patch makes it so that only a configurable number of batchers can
flush simultaneously in the C++ client. The default is 1, making it so
that flushes occur serially, although applies may take place during an
outstanding flush as long as there are available buffers (the default
number of mutation buffers remains 2).
This patch also adds some basic unit testing for this functionality and
enables the previously-failing client op interleave test.
Change-Id: I3e48a054093163824573962963ddcba122a948b8
---
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/integration-tests/client_flush_interleave-itest.cc
6 files changed, 205 insertions(+), 68 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/33/5533/1
--
To view, visit http://gerrit.cloudera.org:8080/5533
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I3e48a054093163824573962963ddcba122a948b8
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Mike Percy <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Todd Lipcon <[email protected]>
