Skip to content

Conversation

@maver1ck
Copy link
Contributor

@maver1ck maver1ck commented Aug 2, 2016

What changes were proposed in this pull request?

This PR add correct support for PPD when using non-vectorized Parquet reader.

How was this patch tested?

Manual performance tests. Existing automatic tests.

@davies
Copy link
Contributor

davies commented Aug 2, 2016

@maver1ck Thanks for sending out this, I'd prefer to merge #13701, there are already lots of discussions there.

@maver1ck
Copy link
Contributor Author

maver1ck commented Aug 2, 2016

@davies
No problem.
I just want to isolate the reason of performance regression.
Feel free to close this PR when #13701 will be merged.

@SparkQA
Copy link

SparkQA commented Aug 2, 2016

Test build #63136 has finished for PR 14465 at commit 652f557.

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

@maver1ck maver1ck changed the title [SPARK-16320][SPARK-16321] Fixing performance regression when reading… [SPARK-16321] Fixing performance regression when reading… Aug 3, 2016
@srowen
Copy link
Member

srowen commented Aug 4, 2016

This should just be closed, no need to wait. It stays around and could even be reopened if needed.

@maver1ck
Copy link
Contributor Author

maver1ck commented Aug 4, 2016

OK.

@maver1ck maver1ck closed this Aug 4, 2016
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