Skip to content

Conversation

@bogdanrdc
Copy link
Contributor

What changes were proposed in this pull request?

This is a small change to make GeneratorOuter always unresolved. It is mostly no-op change but makes it more clear since GeneratorOuter shouldn't survive analysis phase.
This requires also handling in ResolveAliases rule.

How was this patch tested?

Existing generator tests.

@bogdanrdc
Copy link
Contributor Author

cc @rxin @hvanhovell

@SparkQA
Copy link

SparkQA commented Feb 22, 2017

Test build #73276 has finished for PR 17026 at commit c6e8807.

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

@hvanhovell
Copy link
Contributor

LGTM - merging to master. Thanks!

@asfgit asfgit closed this in 10c566c Feb 22, 2017
Yunni pushed a commit to Yunni/spark that referenced this pull request Feb 27, 2017
## What changes were proposed in this pull request?

This is a small change to make GeneratorOuter always unresolved. It is mostly no-op change but makes it more clear since GeneratorOuter shouldn't survive analysis phase.
This requires also handling in ResolveAliases rule.

## How was this patch tested?
Existing generator tests.

Author: Bogdan Raducanu <[email protected]>
Author: Reynold Xin <[email protected]>

Closes apache#17026 from bogdanrdc/PR16958.
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