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.

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

[SPARK-10591][SQL][TEST] Add a testcase to ensure if checkAnswer handles map correctly #13913

Closed
wants to merge 1 commit into from
Closed

Conversation

dongjoon-hyun
Copy link
Member

What changes were proposed in this pull request?

This PR adds a testcase to ensure if checkAnswer handles Map type correctly.

How was this patch tested?

Pass the jenkins tests.

@dongjoon-hyun
Copy link
Member Author

Hi, @liancheng .
When I check SPARK-10591 today, it is handled by Row.equals correctly.
I just make this PR for ensuring that for the future.
If that is not worth a cent, you can close SPARK-10591 issue as Not A Problem.

@SparkQA
Copy link

SparkQA commented Jun 26, 2016

Test build #61265 has finished for PR 13913 at commit 834f515.

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

@dongjoon-hyun dongjoon-hyun changed the title [SPARK-10591][TEST] Add a testcase to ensure if checkAnswer handles map correctly [SPARK-10591][SQL][TEST] Add a testcase to ensure if checkAnswer handles map correctly Jun 26, 2016
asfgit pushed a commit that referenced this pull request Jun 27, 2016
…ndles map correctly

## What changes were proposed in this pull request?

This PR adds a testcase to ensure if `checkAnswer` handles Map type correctly.

## How was this patch tested?

Pass the jenkins tests.

Author: Dongjoon Hyun <dongjoon@apache.org>

Closes #13913 from dongjoon-hyun/SPARK-10591.

(cherry picked from commit 11f420b)
Signed-off-by: Cheng Lian <lian@databricks.com>
@asfgit asfgit closed this in 11f420b Jun 27, 2016
@liancheng
Copy link
Contributor

Thanks! Merged to master and branch-2.0.

@dongjoon-hyun
Copy link
Member Author

Thank you for merging, @liancheng !

@dongjoon-hyun dongjoon-hyun deleted the SPARK-10591 branch July 20, 2016 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants