Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[SPARK-21111] [TEST] [2.2] Fix the test failure of describe.sql #18316

Closed
wants to merge 1 commit into from
Closed

[SPARK-21111] [TEST] [2.2] Fix the test failure of describe.sql #18316

wants to merge 1 commit into from

Conversation

gatorsmile
Copy link
Member

What changes were proposed in this pull request?

Test failed in describe.sql.

We need to fix the related bug introduced in (#17649) in the follow-up PR to master.

How was this patch tested?

N/A

@gatorsmile
Copy link
Member Author

cc @yhuai @cloud-fan @sameeragarwal

@yhuai
Copy link
Contributor

yhuai commented Jun 15, 2017

lgtm

@SparkQA
Copy link

SparkQA commented Jun 15, 2017

Test build #78121 has finished for PR 18316 at commit 0858ec8.

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

@gatorsmile
Copy link
Member Author

retest this please

@cloud-fan
Copy link
Contributor

LGTM

@SparkQA
Copy link

SparkQA commented Jun 16, 2017

Test build #78135 has finished for PR 18316 at commit 0858ec8.

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

@yhuai
Copy link
Contributor

yhuai commented Jun 16, 2017

thanks! merging to branch-2.2

asfgit pushed a commit that referenced this pull request Jun 16, 2017
@gatorsmile @yhuai
[SPARK-21111][TEST][2.2] Fix the test failure of describe.sql
a585c87 
## What changes were proposed in this pull request?
Test failed in `describe.sql`.

We need to fix the related bug introduced in (#17649) in the follow-up PR to master.

## How was this patch tested?
N/A

Author: gatorsmile <gatorsmile@gmail.com>

Closes #18316 from gatorsmile/fix.
@yhuai
Copy link
Contributor

yhuai commented Jun 16, 2017

Thanks! I have merged this pr to branch-2.2.

@gatorsmile gatorsmile closed this Jun 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@gatorsmile @yhuai @SparkQA @cloud-fan