Skip to content

Conversation

@mgaido91
Copy link
Contributor

@mgaido91 mgaido91 commented May 7, 2018

What changes were proposed in this pull request?

SPARK-24160 is causing a compilation failure (after SPARK-24143 was merged). This fixes the issue.

How was this patch tested?

building successfully

@mgaido91
Copy link
Contributor Author

mgaido91 commented May 7, 2018

cc @cloud-fan @JoshRosen @jinxing64

@viirya
Copy link
Member

viirya commented May 7, 2018

LGTM

@cloud-fan
Copy link
Contributor

thanks! I'm merging it to unblock the build, since it already passes the compilation.

Copy link
Member

@HyukjinKwon HyukjinKwon left a comment

Choose a reason for hiding this comment

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

lgtm

@asfgit asfgit closed this in f065280 May 7, 2018
@SparkQA
Copy link

SparkQA commented May 7, 2018

Test build #90300 has finished for PR 21256 at commit d8fecb5.

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

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