Skip to content

Conversation

@nchammas
Copy link
Contributor

  • Remove unused code to get jq
  • Set timeout on tests and report gracefully on them

* Remove unused code to get jq
* Set timeout on tests and report gracefully on them
@SparkQA
Copy link

SparkQA commented Aug 15, 2014

QA tests have started for PR 1974 at commit 279526e.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Aug 15, 2014

Tests timed out after a configured wait of .

@SparkQA
Copy link

SparkQA commented Aug 15, 2014

QA tests have started for PR 1974 at commit 8b1ea41.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Aug 15, 2014

Tests timed out after a configured wait of 1m.

@SparkQA
Copy link

SparkQA commented Aug 15, 2014

QA tests have started for PR 1974 at commit d1f1b6b.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Aug 15, 2014

QA tests have finished for PR 1974 at commit d1f1b6b.

  • This patch passes unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@nchammas nchammas changed the title [SPARK-3076] catch & report test timeouts [SPARK-3076] [Jenkins] catch & report test timeouts Aug 16, 2014
@pwendell
Copy link
Contributor

LGTM - thanks for cleaning up the un-used code also. Let's see how this goes.

@asfgit asfgit closed this in 4bdfaa1 Aug 16, 2014
@pwendell
Copy link
Contributor

I merged this and bumped the timeout here to 120m and bumped the Jenkins-side timeout to 130m

@nchammas
Copy link
Contributor Author

Sounds good!

@nchammas
Copy link
Contributor Author

and bumped the Jenkins-side timeout to 130m

@pwendell We should probably also remove this comment then, since it's now obsolete.

@nchammas
Copy link
Contributor Author

nchammas commented Sep 2, 2014

I merged this and bumped the timeout here to 120m and bumped the Jenkins-side timeout to 130m

Btw @pwendell how did you do that? I'm curious since the commit log doesn't show a separate item for the change you made to set the timeout from 100m (which is how I submitted PR) to 120m.

xiliu82 pushed a commit to xiliu82/spark that referenced this pull request Sep 4, 2014
* Remove unused code to get jq
* Set timeout on tests and report gracefully on them

Author: Nicholas Chammas <[email protected]>

Closes apache#1974 from nchammas/master and squashes the following commits:

d1f1b6b [Nicholas Chammas] set timeout to realistic number
8b1ea41 [Nicholas Chammas] fix formatting
279526e [Nicholas Chammas] [SPARK-3076] catch & report test timeouts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants