You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@impala.apache.org by "Alex Behm (Code Review)" <ge...@cloudera.org> on 2016/04/07 07:54:58 UTC

[Impala-CR](cdh5-trunk) Add 10 minute timeout to all end-to-end tests

Alex Behm has posted comments on this change.

Change subject: Add 10 minute timeout to all end-to-end tests
......................................................................


Patch Set 2:

Casey, I'm thinking that a time based timeout could cause instability. Basically every test that relies on time is flaky, and the timeout would add that to all tests. Also, if there is hang in a test, then we might want to debug it live.

What do you think? Just trying to continue the discussion on this patch.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Id5d4d9b0363eed01b8f52e32a5a8c90592432a29
Gerrit-PatchSet: 2
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Casey Ching <ca...@cloudera.com>
Gerrit-Reviewer: Alex Behm <al...@cloudera.com>
Gerrit-Reviewer: Casey Ching <ca...@cloudera.com>
Gerrit-Reviewer: Martin Grund <gr...@gmail.com>
Gerrit-HasComments: No