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

[kudu-CR] Make fuzz-itest run serially 

Hello Mike Percy,

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

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

to review the following change.

Change subject: Make fuzz-itest run serially
......................................................................

Make fuzz-itest run serially

This test relies on restarting servers, and on the flaky dashboard it
appears to fail often due to port conflicts. Rather than fix it at the
moment, let's just make it run serially. This won't affect precommit
time, since in the dist-test environment, _all_ tests run alone on a
machine.

Change-Id: Ia6e44b4318fec14f6f0cbc8e697fe39049a6b246
---
M src/kudu/integration-tests/CMakeLists.txt
1 file changed, 1 insertion(+), 1 deletion(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia6e44b4318fec14f6f0cbc8e697fe39049a6b246
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Mike Percy <mp...@apache.org>

[kudu-CR] Make fuzz-itest run serially

Posted by "Mike Percy (Code Review)" <ge...@cloudera.org>.
Mike Percy has submitted this change and it was merged.

Change subject: Make fuzz-itest run serially
......................................................................


Make fuzz-itest run serially

This test relies on restarting servers, and on the flaky dashboard it
appears to fail often due to port conflicts. Rather than fix it at the
moment, let's just make it run serially. This won't affect precommit
time, since in the dist-test environment, _all_ tests run alone on a
machine.

Change-Id: Ia6e44b4318fec14f6f0cbc8e697fe39049a6b246
Reviewed-on: http://gerrit.cloudera.org:8080/5398
Tested-by: Kudu Jenkins
Reviewed-by: Mike Percy <mp...@apache.org>
---
M src/kudu/integration-tests/CMakeLists.txt
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Mike Percy: Looks good to me, approved
  Kudu Jenkins: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia6e44b4318fec14f6f0cbc8e697fe39049a6b246
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mp...@apache.org>

[kudu-CR] Make fuzz-itest run serially

Posted by "Mike Percy (Code Review)" <ge...@cloudera.org>.
Mike Percy has posted comments on this change.

Change subject: Make fuzz-itest run serially
......................................................................


Patch Set 1: Code-Review+2

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia6e44b4318fec14f6f0cbc8e697fe39049a6b246
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-HasComments: No