impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alex Behm (Code Review)" <ger...@cloudera.org>
Subject [Impala-CR](cdh5-trunk) Add 10 minute timeout to all end-to-end tests
Date Thu, 07 Apr 2016 05:54:58 GMT
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 <casey@cloudera.com>
Gerrit-Reviewer: Alex Behm <alex.behm@cloudera.com>
Gerrit-Reviewer: Casey Ching <casey@cloudera.com>
Gerrit-Reviewer: Martin Grund <grundprinzip@gmail.com>
Gerrit-HasComments: No

Mime
View raw message