[SPARK-12882][SQL] simplify bucket tests and add more comments #10813

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

Closed

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

Contributor

cloud-fan commented Jan 18, 2016

Right now, the bucket tests are kind of hard to understand, this PR simplifies them and add more commetns.


          simplify bucket tests and add more comments

436b349

Contributor Author

cloud-fan commented Jan 18, 2016

cc @rxin @yhuai

SparkQA commented Jan 18, 2016

Test build #49613 has finished for PR 10813 at commit 436b349.

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

Contributor

rxin commented Jan 18, 2016

I've merged this. Thanks.

asfgit closed this in

cloud-fan deleted the bucket-comment branch

January 18, 2016 23:21

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

Labels

None yet