[kudu-CR] delete table-test: bump timeout waiting for tablet to start

2016-07-20 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged.

Change subject: delete_table-test: bump timeout waiting for tablet to start
..


delete_table-test: bump timeout waiting for tablet to start

It seems like TestAutoTombstoneAfterRemoteBootstrapRemoteFails can fail because
it times out waiting for the tablet to restart, particularly in TSAN where
things run slowly. Just bumping the timeout to 60s vs 30s.

Change-Id: I6a3798483f506dd4cee2c685c4eff54c5df6569a
Reviewed-on: http://gerrit.cloudera.org:8080/3689
Tested-by: Kudu Jenkins
Reviewed-by: Todd Lipcon 
---
M src/kudu/integration-tests/delete_table-test.cc
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Todd Lipcon: Looks good to me, approved
  Kudu Jenkins: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I6a3798483f506dd4cee2c685c4eff54c5df6569a
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy 
Gerrit-Reviewer: Todd Lipcon 


[kudu-CR] delete table-test: bump timeout waiting for tablet to start

2016-07-20 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change.

Change subject: delete_table-test: bump timeout waiting for tablet to start
..


Patch Set 1: Code-Review+2

self-+2ing trivial patch

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I6a3798483f506dd4cee2c685c4eff54c5df6569a
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy 
Gerrit-Reviewer: Todd Lipcon 
Gerrit-HasComments: No


[kudu-CR] delete table-test: bump timeout waiting for tablet to start

2016-07-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change.

Change subject: delete_table-test: bump timeout waiting for tablet to start
..


Patch Set 1:

Build Started http://104.196.14.100/job/kudu-gerrit/2569/

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I6a3798483f506dd4cee2c685c4eff54c5df6569a
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy 
Gerrit-HasComments: No


[kudu-CR] delete table-test: bump timeout waiting for tablet to start

2016-07-20 Thread Todd Lipcon (Code Review)
Hello Mike Percy,

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

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

to review the following change.

Change subject: delete_table-test: bump timeout waiting for tablet to start
..

delete_table-test: bump timeout waiting for tablet to start

It seems like TestAutoTombstoneAfterRemoteBootstrapRemoteFails can fail because
it times out waiting for the tablet to restart, particularly in TSAN where
things run slowly. Just bumping the timeout to 60s vs 30s.

Change-Id: I6a3798483f506dd4cee2c685c4eff54c5df6569a
---
M src/kudu/integration-tests/delete_table-test.cc
1 file changed, 1 insertion(+), 1 deletion(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I6a3798483f506dd4cee2c685c4eff54c5df6569a
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon 
Gerrit-Reviewer: Mike Percy