Skip to content

Conversation

@ueshin
Copy link
Member

@ueshin ueshin commented Mar 29, 2017

What changes were proposed in this pull request?

Commit 6c70a38 broke the build for scala 2.10. The commit uses some reflections which are not available in Scala 2.10. This PR fixes them.

How was this patch tested?

Existing tests.

@ueshin
Copy link
Member Author

ueshin commented Mar 29, 2017

cc @cloud-fan @michalsenkyr

@SparkQA
Copy link

SparkQA commented Mar 30, 2017

Test build #75371 has finished for PR 17473 at commit 36d12fd.

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

@cloud-fan
Copy link
Contributor

LGTM, thanks!

@ueshin
Copy link
Member Author

ueshin commented Mar 30, 2017

Merging to master!

@asfgit asfgit closed this in dd2e7d5 Mar 30, 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