Skip to content

Conversation

@gatorsmile
Copy link
Member

@gatorsmile gatorsmile commented Jun 16, 2017

@gatorsmile gatorsmile changed the title [SPARK-21114] [TEST] [2.1] Fix test failure fix in Spark 2.1 due to name mismatch [SPARK-21114] [TEST] [2.1] Fix test failure in Spark 2.1 due to name mismatch Jun 16, 2017
@gatorsmile
Copy link
Member Author

cc @cloud-fan @yhuai @sameeragarwal

@gatorsmile gatorsmile changed the title [SPARK-21114] [TEST] [2.1] Fix test failure in Spark 2.1 due to name mismatch [SPARK-21114] [TEST] [2.1] Fix test failure in Spark 2.1/2.0 due to name mismatch Jun 16, 2017
@gatorsmile
Copy link
Member Author

We need to backport it to 2.0 too.

@cloud-fan
Copy link
Contributor

LGTM, seems PR that touches sql text tests are very hard to backport, we should always send new PRs.

@SparkQA
Copy link

SparkQA commented Jun 16, 2017

Test build #78143 has finished for PR 18319 at commit 367e8e5.

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

@cloud-fan
Copy link
Contributor

thanks, merging to 2.1

asfgit pushed a commit that referenced this pull request Jun 16, 2017
…e mismatch

## What changes were proposed in this pull request?
Name mismatch between 2.1/2.0 and 2.2. Thus, the test cases failed after we backport a fix to 2.1/2.0. This PR is to fix the issue.

https://amplab.cs.berkeley.edu/jenkins/view/Spark%20QA%20Test/job/spark-branch-2.1-test-maven-hadoop-2.7/lastCompletedBuild/testReport/org.apache.spark.sql/SQLQueryTestSuite/arithmetic_sql/

https://amplab.cs.berkeley.edu/jenkins/view/Spark%20QA%20Test/job/spark-branch-2.0-test-maven-hadoop-2.2/lastCompletedBuild/testReport/org.apache.spark.sql/SQLQueryTestSuite/arithmetic_sql/

## How was this patch tested?
N/A

Author: gatorsmile <gatorsmile@gmail.com>

Closes #18319 from gatorsmile/fixDecimal.
@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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants