Skip to content

Conversation

@rxin
Copy link
Contributor

@rxin rxin commented Mar 16, 2017

What changes were proposed in this pull request?

This is a tiny teeny refactoring to pass data filters also to the FileIndex, so FileIndex can have a more global view on predicates.

How was this patch tested?

Change should be covered by existing test cases.

@SparkQA
Copy link

SparkQA commented Mar 16, 2017

Test build #74687 has finished for PR 17322 at commit 87a8be5.

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

@gatorsmile
Copy link
Member

LGTM pending Jenkins

@SparkQA
Copy link

SparkQA commented Mar 17, 2017

Test build #74696 has finished for PR 17322 at commit a667f80.

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

@gatorsmile
Copy link
Member

Thanks! 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