New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Minor] [SQL] Remove Useless zipWithIndex from ResolveAliases #20009

Closed

gatorsmile wants to merge 1 commit into apache:master from gatorsmile:try22

Member

gatorsmile commented Dec 18, 2017

What changes were proposed in this pull request?

Remove useless zipWithIndex from ResolveAliases .

How was this patch tested?

The existing tests


fix.

4ec9a37

gatorsmile changed the title ~~[Minor] [SQL] Cleanup ResolveAliases~~ [Minor] [SQL] Remove Useless zipWithIndex from ResolveAliases

SparkQA commented Dec 18, 2017

Test build #85068 has finished for PR 20009 at commit 4ec9a37.

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

Member Author

gatorsmile commented Dec 18, 2017

Contributor

cloud-fan commented Dec 19, 2017

LGTM, merging to master!

asfgit closed this in

d4e6959

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment