Skip to content

Conversation

@rxin
Copy link
Contributor

@rxin rxin commented Jul 6, 2016

What changes were proposed in this pull request?

This patch removes InSet filter pushdown from Parquet data source, since row-based pushdown is not beneficial to Spark and brings extra complexity to the code base.

How was this patch tested?

N/A

@rxin
Copy link
Contributor Author

rxin commented Jul 6, 2016

cc @liancheng

@SparkQA
Copy link

SparkQA commented Jul 6, 2016

Test build #61865 has finished for PR 14076 at commit 977ff10.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Jul 7, 2016

Test build #61876 has finished for PR 14076 at commit eb26650.

  • This patch passes all tests.
  • This patch does not merge cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Jul 7, 2016

Test build #61877 has finished for PR 14076 at commit fbdcd7b.

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

@liancheng
Copy link
Contributor

LGTM. Merging to master.

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.

3 participants