Alexey Serbin has posted comments on this change. Change subject: [client-test] added basic test for RWYW behavior ......................................................................
Patch Set 2: (1 comment) > As I had suggested if you made so that the test always failed, we > could merge it disabled and then re-enable it when we get this > stuff fixed. I think we can abandon this change if it does not provide good guarantees on exercising RYW behavior. Let's figure out what behavior we want from more elaborate test and put that into the pool of integration tests. http://gerrit.cloudera.org:8080/#/c/4583/2/src/kudu/client/client-test.cc File src/kudu/client/client-test.cc: The patchset 2 is just to provide an example for the case I mentioned in the comment for item '4. Potential unrepeatable read' at https://docs.google.com/document/d/1EaKlJyQdMBz6G-Xn5uktY-d_x0uRmjMCrDGP5rZ7AoI -- To view, visit http://gerrit.cloudera.org:8080/4583 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3c237a41a1434199848ac5fc978fab27e565e2d8 Gerrit-PatchSet: 2 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: Tidy Bot Gerrit-HasComments: Yes
