Skip to content

[SPARK-23931][SQL][FOLLOW-UP] Make arrays_zip in function.scala @scala.annotation.varargs.#21630

Closed
ueshin wants to merge 1 commit into
apache:masterfrom
ueshin:issues/SPARK-23931/fup1
Closed

[SPARK-23931][SQL][FOLLOW-UP] Make arrays_zip in function.scala @scala.annotation.varargs.#21630
ueshin wants to merge 1 commit into
apache:masterfrom
ueshin:issues/SPARK-23931/fup1

Conversation

@ueshin

@ueshin ueshin commented Jun 25, 2018

Copy link
Copy Markdown
Member

What changes were proposed in this pull request?

This is a follow-up pr of #21045 which added arrays_zip.
The arrays_zip in functions.scala should've been @scala.annotation.varargs.

This pr makes it @scala.annotation.varargs.

How was this patch tested?

Existing tests.

@ueshin

ueshin commented Jun 25, 2018

Copy link
Copy Markdown
Member Author

cc @DylanGuedes @HyukjinKwon @viirya @mgaido91

@HyukjinKwon HyukjinKwon left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@viirya

viirya commented Jun 25, 2018 via email

Copy link
Copy Markdown
Member

@SparkQA

SparkQA commented Jun 25, 2018

Copy link
Copy Markdown

Test build #92279 has finished for PR 21630 at commit 7e0aa18.

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

@gatorsmile gatorsmile left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

Thanks! Merged to master.

@asfgit asfgit closed this in 6e0596e Jun 25, 2018
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.

5 participants