[SQL][minor] use stricter type parameter to make it clear that parquet reader returns UnsafeRow #14458

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:parquet

Contributor

cloud-fan commented Aug 2, 2016

What changes were proposed in this pull request?

a small code style change, it's better to make the type parameter more accurate.

How was this patch tested?

N/A


          use stricter type parameter to make it clear that parquet reader retu…

67a4c17

…rns UnsafeRow

Contributor Author

cloud-fan commented Aug 2, 2016

cc @clockfly @yhuai @liancheng

SparkQA commented Aug 2, 2016

Test build #63115 has finished for PR 14458 at commit 67a4c17.

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

Contributor

liancheng commented Aug 3, 2016

LGTM, merging to master.

asfgit closed this in

ae22628

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

Labels

None yet