You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kudu.apache.org by "Todd Lipcon (Code Review)" <ge...@cloudera.org> on 2016/02/18 02:40:14 UTC

[kudu-CR] build: fix ignoring of clock sync errors in flaky test list

Hello Adar Dembo,

I'd like you to do a code review.  Please visit

    http://gerrit.cloudera.org:8080/2218

to review the following change.

Change subject: build: fix ignoring of clock sync errors in flaky test list
......................................................................

build: fix ignoring of clock sync errors in flaky test list

Previously, I tried to add this same functionality, but used 'grep' instead of
'zgrep'. In the production test slaves, the test logs are gzipped, so we need
to 'zgrep'. It turns out that zgrep is also smart enough to grep non-compressed
files, so it's safe to use it regardless of the file extension.

Change-Id: Ic8645f6f046700a28d3306e0db1be9dbfb06a433
---
M build-support/report-test.sh
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/18/2218/1
-- 
To view, visit http://gerrit.cloudera.org:8080/2218
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic8645f6f046700a28d3306e0db1be9dbfb06a433
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>