Skip to content

Conversation

@hvanhovell
Copy link
Contributor

@hvanhovell hvanhovell commented Nov 4, 2016

What changes were proposed in this pull request?

This PR backports two subquery related PRs to branch-2.0:

How was this patch tested?

Added a tests to SubquerySuite.

hvanhovell and others added 2 commits November 4, 2016 21:33
…edicate subqueries

## What changes were proposed in this pull request?
The `PushDownPredicate` rule can create a wrong result if we try to push a filter containing a predicate subquery through a project when the subquery and the project share attributes (have the same source).

The current PR fixes this by making sure that we do not push down when there is a predicate subquery that outputs the same attributes as the filters new child plan.

## How was this patch tested?
Added a test to `SubquerySuite`. nsyca has done previous work this. I have taken test from his initial PR.
@hvanhovell hvanhovell changed the title [SPARK-17337][SQL] Do not pushdown predicates through filters with predicate subqueries [BRANCH-2.0] [SPARK-17337][SPARK-16804][SQL] Backport subquery related PRs [BRANCH-2.0] Nov 4, 2016
@rxin
Copy link
Contributor

rxin commented Nov 4, 2016

LGTM pending Jenkins.

@SparkQA
Copy link

SparkQA commented Nov 4, 2016

Test build #68157 has finished for PR 15772 at commit 898f722.

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

@rxin
Copy link
Contributor

rxin commented Nov 4, 2016

Merging in.

asfgit pushed a commit that referenced this pull request Nov 4, 2016
… PRs

## What changes were proposed in this pull request?
This PR backports two subquery related PRs to branch-2.0:

- #14411
- #15761

## How was this patch tested?
Added a tests  to `SubquerySuite`.

Author: Nattavut Sutyanyong <nsy.can@gmail.com>
Author: Herman van Hovell <hvanhovell@databricks.com>

Closes #15772 from hvanhovell/SPARK-17337-2.0.
@hvanhovell hvanhovell closed this Nov 4, 2016
@SparkQA
Copy link

SparkQA commented Nov 5, 2016

Test build #68164 has finished for PR 15772 at commit 7aec0b2.

  • 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