Skip to content

[SPARK-18167] [SQL] Also log all partitions when the SQLQuerySuite test flakes - #15701

Closed
ericl wants to merge 1 commit into
apache:masterfrom
ericl:print-more-info
Closed

[SPARK-18167] [SQL] Also log all partitions when the SQLQuerySuite test flakes#15701
ericl wants to merge 1 commit into
apache:masterfrom
ericl:print-more-info

Conversation

@ericl

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

One possibility for this test flaking is that we have corrupted the partition schema somehow in the tests, which causes the cast to decimal to fail in the call. This should at least show us the actual partition values.

How was this patch tested?

Run it locally, it prints out something like ArrayBuffer(test(partcol=0), test(partcol=1), test(partcol=2), test(partcol=3), test(partcol=4)).

@ericl

Copy link
Copy Markdown
ContributorAuthor

@yhuai

@yhuai

Copy link
Copy Markdown
Contributor

lgtm

@SparkQA

Copy link
Copy Markdown

Test build #67836 has finished for PR 15701 at commit 767fef8.

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

@yhuai

Copy link
Copy Markdown
Contributor

merging to master. Thanks

uzadude pushed a commit to uzadude/spark that referenced this pull request Jan 27, 2017
…t flakes
## What changes were proposed in this pull request?
One possibility for this test flaking is that we have corrupted the partition schema somehow in the tests, which causes the cast to decimal to fail in the call. This should at least show us the actual partition values.
## How was this patch tested?
Run it locally, it prints out something like `ArrayBuffer(test(partcol=0), test(partcol=1), test(partcol=2), test(partcol=3), test(partcol=4))`.
Author: Eric Liang <ekl@databricks.com>
Closesapache#15701 from ericl/print-more-info.
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

@ericl@yhuai@SparkQA