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

Jump to bottom

[SPARK-22076][SQL][followup] Expand.projections should not be a Stream #19298

Closed

cloud-fan wants to merge 1 commit into apache:master from cloud-fan:bug

Contributor

cloud-fan commented Sep 21, 2017

What changes were proposed in this pull request?

This a follow-up of #19289 , we missed another place: rollup. Seq.init.toSeq also returns a Stream, we should fix it too.

How was this patch tested?

manually


          Expand.projections should not be a Stream

aa263d7

Contributor Author

cloud-fan commented Sep 21, 2017

SparkQA commented Sep 21, 2017

Test build #82014 has finished for PR 19298 at commit aa263d7.

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

gatorsmile approved these changes

View reviewed changes

Member

gatorsmile commented Sep 21, 2017

LGTM

Merging to master/2.2

asfgit closed this in

352bea5

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

Labels

None yet