Alexey Serbin has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/17120 )

Change subject: [client-test] added WriteWhileRestartingMultipleTabletServers
......................................................................

[client-test] added WriteWhileRestartingMultipleTabletServers

While working on tests for multi-row transactions, I added a similar
scenario and found it a bit flaky.  So, I wanted to see if the same
is true for the non-transactional write operations.  It turns out
the latter is pretty stable, so I need to dig in to find the root
cause of the former.  Anyways, I think this is a good scenario to
add into client-test.cc, extending already existing scenario
ClientTest.TestWriteWhileRestarting to multi-replica case and going
through multiple restarts.  Another important detail is that the newly
added test scenario verifies the number of persisted rows in the end.

I also did a small touch-up of the code related to the utility method
ClientTest::CountRowsFromClient().

Change-Id: I95d1456dea2e6e2bb7d8b0c5d05e95798098710d
Reviewed-on: http://gerrit.cloudera.org:8080/17120
Tested-by: Alexey Serbin <aser...@cloudera.com>
Reviewed-by: Andrew Wong <aw...@cloudera.com>
---
M src/kudu/client/client-test.cc
1 file changed, 105 insertions(+), 42 deletions(-)

Approvals:
  Alexey Serbin: Verified
  Andrew Wong: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/17120
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I95d1456dea2e6e2bb7d8b0c5d05e95798098710d
Gerrit-Change-Number: 17120
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Hao Hao <hao....@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to