Skip to content

Conversation

@sameeragarwal
Copy link
Member

What changes were proposed in this pull request?

This PR is a small follow up on #11338 (https://issues.apache.org/jira/browse/SPARK-13092) to use ExpressionSet as part of the verification logic in ConstraintPropagationSuite.

How was this patch tested?

No new tests added. Just changes the verification logic in ConstraintPropagationSuite.

@sameeragarwal
Copy link
Member Author

cc @marmbrus @yhuai

@marmbrus
Copy link
Contributor

marmbrus commented Mar 9, 2016

LGTM pending tests

@SparkQA
Copy link

SparkQA commented Mar 9, 2016

Test build #52765 has finished for PR 11611 at commit 1872334.

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

@marmbrus
Copy link
Contributor

marmbrus commented Mar 9, 2016

Merged to master.

@asfgit asfgit closed this in dbf2a7c Mar 9, 2016
roygao94 pushed a commit to roygao94/spark that referenced this pull request Mar 22, 2016
## What changes were proposed in this pull request?

This PR is a small follow up on apache#11338 (https://issues.apache.org/jira/browse/SPARK-13092) to use `ExpressionSet` as part of the verification logic in `ConstraintPropagationSuite`.
## How was this patch tested?

No new tests added. Just changes the verification logic in `ConstraintPropagationSuite`.

Author: Sameer Agarwal <[email protected]>

Closes apache#11611 from sameeragarwal/expression-set.
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