You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Adar Dembo (Code Review)" <ge...@cloudera.org> on 2016/09/15 20:46:28 UTC

[kudu-CR] webserver-stress-itest: increase timeouts

Hello Todd Lipcon,

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

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

to review the following change.

Change subject: webserver-stress-itest: increase timeouts
......................................................................

webserver-stress-itest: increase timeouts

The flaky test dashboard shows a lot of ASAN and TSAN failures while waiting
for tablets to start running, so let's drastically increase that timeout.

Additionally, some ASAN builds timed out while creating the test table, so
let's also reduce the number of checkers run in ASAN.

Change-Id: I7fb042be33af0b4ecbdc39af471eab3e1c872d53
---
M src/kudu/integration-tests/webserver-stress-itest.cc
1 file changed, 3 insertions(+), 3 deletions(-)


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

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

[kudu-CR] webserver-stress-itest: increase timeouts

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

Change subject: webserver-stress-itest: increase timeouts
......................................................................


webserver-stress-itest: increase timeouts

The flaky test dashboard shows a lot of ASAN and TSAN failures while waiting
for tablets to start running, so let's drastically increase that timeout.

Additionally, some ASAN builds timed out while creating the test table, so
let's also reduce the number of checkers run in ASAN.

Change-Id: I7fb042be33af0b4ecbdc39af471eab3e1c872d53
Reviewed-on: http://gerrit.cloudera.org:8080/4427
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin <as...@cloudera.com>
---
M src/kudu/integration-tests/webserver-stress-itest.cc
1 file changed, 3 insertions(+), 3 deletions(-)

Approvals:
  Alexey Serbin: Looks good to me, approved
  Kudu Jenkins: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I7fb042be33af0b4ecbdc39af471eab3e1c872d53
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>

[kudu-CR] webserver-stress-itest: increase timeouts

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

Change subject: webserver-stress-itest: increase timeouts
......................................................................


Patch Set 1: Code-Review+2

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7fb042be33af0b4ecbdc39af471eab3e1c872d53
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-HasComments: No