Skip to content

[SPARK-11738] [SQL] Making ArrayType orderable - #9718

Closed
yhuai wants to merge 6 commits into
apache:masterfrom
yhuai:makingArrayOrderable
Closed

[SPARK-11738] [SQL] Making ArrayType orderable#9718
yhuai wants to merge 6 commits into
apache:masterfrom
yhuai:makingArrayOrderable

Conversation

@yhuai

Copy link
Copy Markdown
Contributor

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These lines have been moved to compareValue.

@yhuai

Copy link
Copy Markdown
ContributorAuthor

@davies take a look?

@SparkQA

Copy link
Copy Markdown

Test build #45940 has finished for PR 9718 at commit f5f074d.

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

@SparkQA

Copy link
Copy Markdown

Test build #45945 has finished for PR 9718 at commit f43a7f9.

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

@SparkQA

Copy link
Copy Markdown

Test build #45946 has finished for PR 9718 at commit 7228093.

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These could be defined in the loop (let compiler to optimize them easily)

@davies

Copy link
Copy Markdown
Contributor

LGTM, and some minor comments

@SparkQA

Copy link
Copy Markdown

Test build #45955 has finished for PR 9718 at commit aadfeab.

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

@davies

Copy link
Copy Markdown
Contributor

Merged into master and 1.6 branch, thanks!

asfgit pushed a commit that referenced this pull request Nov 15, 2015
https://issues.apache.org/jira/browse/SPARK-11738
Author: Yin Huai <yhuai@databricks.com>
Closes#9718 from yhuai/makingArrayOrderable.
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

@yhuai@SparkQA@davies