Skip to content

[SPARK-18167] [SQL] Retry when the SQLQuerySuite test flakes#15708

Closed
ericl wants to merge 1 commit into
apache:masterfrom
ericl:spark-18167-3
Closed

[SPARK-18167] [SQL] Retry when the SQLQuerySuite test flakes#15708
ericl wants to merge 1 commit into
apache:masterfrom
ericl:spark-18167-3

Conversation

@ericl

@ericl ericl commented Nov 1, 2016

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

This will re-run the flaky test a few times after it fails. This will help determine if it's due to nondeterministic test setup, or because of some environment issue (e.g. leaked config from another test).

cc @yhuai

@SparkQA

SparkQA commented Nov 1, 2016

Copy link
Copy Markdown

Test build #67854 has finished for PR 15708 at commit 641337b.

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

@rxin

rxin commented Nov 1, 2016

Copy link
Copy Markdown
Contributor

Merging in master.

@asfgit asfgit closed this in 7d6c871 Nov 1, 2016
@ericl

ericl commented Nov 1, 2016

Copy link
Copy Markdown
Contributor Author

Ok interesting, it seems the test repeatedly fails. This probably means it is a suite ordering issue.

@HyukjinKwon

HyukjinKwon commented Nov 1, 2016

Copy link
Copy Markdown
Member

Hi @ericl, I just happened to come here while tracking down the flaky tests (I met this several times in series and I thought we need a hotfix). I just wonder if it'd make sense to just ignore this test maybe for now?

uzadude pushed a commit to uzadude/spark that referenced this pull request Jan 27, 2017
## What changes were proposed in this pull request?

This will re-run the flaky test a few times after it fails. This will help determine if it's due to nondeterministic test setup, or because of some environment issue (e.g. leaked config from another test).

cc yhuai

Author: Eric Liang <ekl@databricks.com>

Closes apache#15708 from ericl/spark-18167-3.
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.

4 participants