Hello Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/13397
to look at the new patch set (#2).
Change subject: [java] Fix TestHybridTime
......................................................................
[java] Fix TestHybridTime
TestHybridTime tests client propagated timestamps when timestamp
assignment needs to increment the logical clock. However, sometimes
there are jumps in the logical values of the logical clock, which made
this test flaky because it counted on seeing specific numbers of rows at
"magic number" logical values. This adjusts the test to track the
logical values back-propagated from the server and tests the expected
state of the table against those values instead of constants.
I also simplified the test.
I ran the original version 1000 times in dist-test and saw 81 failures.
I ran the new version 1200 times and saw 0 failures.
Change-Id: I98a5bccf2fb366004659f3e6169b5f57f7ced485
---
M java/kudu-client/src/test/java/org/apache/kudu/client/TestHybridTime.java
1 file changed, 63 insertions(+), 78 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/97/13397/2
--
To view, visit http://gerrit.cloudera.org:8080/13397
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I98a5bccf2fb366004659f3e6169b5f57f7ced485
Gerrit-Change-Number: 13397
Gerrit-PatchSet: 2
Gerrit-Owner: Will Berkeley <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Will Berkeley <[email protected]>
