Skip to content

Conversation

@viirya
Copy link
Member

@viirya viirya commented Apr 13, 2018

What changes were proposed in this pull request?

Just found MultiAlias is a CodegenFallback. It should not be as looks like MultiAlias won't be evaluated.

How was this patch tested?

Existing tests.

@SparkQA
Copy link

SparkQA commented Apr 13, 2018

Test build #89341 has finished for PR 21065 at commit 05a57cb.

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

@viirya
Copy link
Member Author

viirya commented Apr 13, 2018

cc @cloud-fan @hvanhovell

@cloud-fan
Copy link
Contributor

LGTM, merging to master!

@asfgit asfgit closed this in 73f2853 Apr 14, 2018
@viirya viirya deleted the multialias-without-codegenfallback branch December 27, 2023 18:35
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