Skip to content

[SPARK-8508] [SQL] Ignores a test case to cleanup unnecessary testing output until #6882 is merged - #6925

Closed
liancheng wants to merge 1 commit into
apache:masterfrom
liancheng:spark-8508
Closed

[SPARK-8508] [SQL] Ignores a test case to cleanup unnecessary testing output until #6882 is merged#6925
liancheng wants to merge 1 commit into
apache:masterfrom
liancheng:spark-8508

Conversation

@liancheng

Copy link
Copy Markdown
Contributor

Currently the test case for SPARK-7862 writes 100,000 lines of integer triples to stderr and makes Jenkins build output unnecessarily large and it's hard to debug other build errors. A proper fix is on the way in #6882. This PR ignores this test case temporarily until #6882 is merged.

@liancheng

Copy link
Copy Markdown
ContributorAuthor

@chenghao-intel@zhichao-li I think this change should be safe, but would you mind to help confirming?

@SparkQA

Copy link
Copy Markdown

Test build #35383 has finished for PR 6925 at commit 239bde7.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • class Module(object):

@liancheng

Copy link
Copy Markdown
ContributorAuthor

retest this please

@SparkQA

Copy link
Copy Markdown

Test build #35385 has finished for PR 6925 at commit 239bde7.

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

@chenghao-intel

Copy link
Copy Markdown
Contributor

Oh, sorry @liancheng actually I've created a PR for this fixing #6882.
If the output size is too small, we couldn't guarantee the deadlock would happen(in unit test), that's why we set it as 100000.

@liancheng

Copy link
Copy Markdown
ContributorAuthor

I see. Since your fix is on the way, I'll just ignore this test temporarily until yours is merged.

@lianchengliancheng changed the title [SPARK-8508] [SQL] Uses smaller scale of test data to reduce unnecessary testing output[SPARK-8508] [SQL] Ignores a test case to cleanup unnecessary testing output until #6882 is mergedJun 21, 2015
@SparkQA

Copy link
Copy Markdown

Test build #35402 has finished for PR 6925 at commit 41e5b47.

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

@liancheng

Copy link
Copy Markdown
ContributorAuthor

Merging to master.

@liancheng
liancheng deleted the spark-8508 branch July 9, 2015 21:34
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

@liancheng@SparkQA@chenghao-intel