Hello Zoltan Chovan, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/23529
to look at the new patch set (#2).
Change subject: KUDU-1973 Enable some unit tests for heartbeat batching.
......................................................................
KUDU-1973 Enable some unit tests for heartbeat batching.
The proper way to test heartbeat batching would be to run all
unit tests using Raft consensus with batching enabled as well.
However, this would nearly double the cost of running the tests,
since most of our long-running tests use a cluster, and therefore
exercise the batching logic.
That said, it also should not be left completely untested.
I think the best approach is to enable a few handpicked tests
to run with batching enabled too. I have selected a few.
For any Raft-related changes in the future, we should run
tests with --enable_multi_raft_heartbeat_batcher=1.
Change-Id: Ifd442b36b46c038fbf836f48feaa21b1ab296d82
---
M src/kudu/integration-tests/raft_consensus-itest.cc
1 file changed, 37 insertions(+), 6 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/29/23529/2
--
To view, visit http://gerrit.cloudera.org:8080/23529
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ifd442b36b46c038fbf836f48feaa21b1ab296d82
Gerrit-Change-Number: 23529
Gerrit-PatchSet: 2
Gerrit-Owner: Zoltan Martonka <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Chovan <[email protected]>
