Skip to content

Conversation

@viirya
Copy link
Member

@viirya viirya commented Dec 31, 2015

A following pr for #9712. Move the test for arrayOfUDT.

Copy link
Contributor

Choose a reason for hiding this comment

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

can we move it to the array section? i.e. the encodeDecodeTest that has a lot of arrayOfxxx

Copy link
Member Author

Choose a reason for hiding this comment

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

ok. it is more proper.

@cloud-fan
Copy link
Contributor

LGTM, pending tests.

@SparkQA
Copy link

SparkQA commented Dec 31, 2015

Test build #48539 has finished for PR 10538 at commit a3e639d.

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

@SparkQA
Copy link

SparkQA commented Dec 31, 2015

Test build #48541 has finished for PR 10538 at commit 8d10703.

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

@rxin
Copy link
Contributor

rxin commented Jan 1, 2016

Thanks - I've merged this.

@asfgit asfgit closed this in c9dbfcc Jan 1, 2016
marmbrus pushed a commit to marmbrus/spark that referenced this pull request Jan 7, 2016
A following pr for apache#9712. Move the test for arrayOfUDT.

Author: Liang-Chi Hsieh <[email protected]>

Closes apache#10538 from viirya/move-udt-test.
@viirya viirya deleted the move-udt-test branch December 27, 2023 18:32
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.

4 participants