Adar Dembo has posted comments on this change.

Change subject: KUDU-2119. Fix failure in encoding-test
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/7967/1//COMMIT_MSG
Commit Message:

Line 16: The fix just replaces any '\0' characters in the random format string.
Does this also fix 
http://dist-test.cloudera.org:8080/diagnose?key=a32eeefe-906c-11e7-8752-0242ac11000f?
 That's a slightly different failure:

  [ RUN      ] TestEncoding.TestBinaryPrefixBlockBuilderRoundTrip
  I0902 22:56:26.483465  8078 test_util.cc:195] Using random seed: -1483122939
  I0902 22:56:26.483530  8078 encoding-test.cc:288] Block: 000000: 1000 3930 
1000 0000 0000                ..90......
  /data/jenkins-workspace/kudu-workspace/src/kudu/cfile/encoding-test.cc:295: 
Failure
  Failed
  Bad status: Not found: no keys in data block


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic46d2a993235e560475d931c7b023eb5b4faf7c2
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Will Berkeley <[email protected]>
Gerrit-HasComments: Yes

Reply via email to