Skip to content

[SPARK-17534] [TESTS] Increase timeouts for DirectKafkaStreamSuite tests - #15094

Closed
a-roberts wants to merge 1 commit into
apache:masterfrom
a-roberts:patch-6
Closed

[SPARK-17534] [TESTS] Increase timeouts for DirectKafkaStreamSuite tests#15094
a-roberts wants to merge 1 commit into
apache:masterfrom
a-roberts:patch-6

Conversation

@a-roberts

Copy link
Copy Markdown
Contributor

## What changes were proposed in this pull request?
There are two tests in this suite that are particularly flaky on the following hardware:

2x Intel(R) Xeon(R) CPU E5-2697 v2 @ 2.70GHz and 16 GB of RAM, 1 TB HDD

This simple PR increases the timeout times and batch duration so they can reliably pass

## How was this patch tested?
Existing unit tests with the two core box where I was seeing the failures often

There are two tests in this suite that are particularly flaky on the following hardware:
2x Intel(R) Xeon(R) CPU E5-2697 v2 @ 2.70GHz and 16 GB of RAM, 1 TB HDD
This simple PR increases the timeout times and batch duration so they can reliably pass
@SparkQA

Copy link
Copy Markdown

Test build #65367 has finished for PR 15094 at commit 9ddecc6.

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

@srowen

Copy link
Copy Markdown
Member

I don't feel strongly about it, but you could write these times in units of seconds now.

@srowen

Copy link
Copy Markdown
Member

Merged to master

wgtmac pushed a commit to wgtmac/spark that referenced this pull request Sep 19, 2016
**## What changes were proposed in this pull request?**
There are two tests in this suite that are particularly flaky on the following hardware:
2x Intel(R) Xeon(R) CPU E5-2697 v2 2.70GHz and 16 GB of RAM, 1 TB HDD
This simple PR increases the timeout times and batch duration so they can reliably pass
**## How was this patch tested?**
Existing unit tests with the two core box where I was seeing the failures often
Author: Adam Roberts <aroberts@uk.ibm.com>
Closesapache#15094 from a-roberts/patch-6.
Sign up for freeto 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

@a-roberts@SparkQA@srowen