Skip to content

Conversation

@yjshen
Copy link
Member

@yjshen yjshen commented Jul 20, 2015

@rxin
Copy link
Contributor

rxin commented Jul 20, 2015

Jenkins, test this please.

cc @cloud-fan

@SparkQA
Copy link

SparkQA commented Jul 20, 2015

Test build #1120 has finished for PR 7540 at commit 91741c1.

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

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SetOperationPushDown

@rxin
Copy link
Contributor

rxin commented Jul 21, 2015

@yjshen when you rename UnionPushDown, make sure you also rename the test suite, and make the test suite file name matches the class name.

@yjshen
Copy link
Member Author

yjshen commented Jul 21, 2015

OK, will do soon.

@rxin
Copy link
Contributor

rxin commented Jul 21, 2015

LGTM.

@rxin
Copy link
Contributor

rxin commented Jul 21, 2015

Thanks - I'm going to merge this.

@asfgit asfgit closed this in ae23059 Jul 21, 2015
@SparkQA
Copy link

SparkQA commented Jul 21, 2015

Test build #1141 has finished for PR 7540 at commit 91741c1.

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

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