Skip to content

Conversation

@aokolnychyi
Copy link
Contributor

What changes were proposed in this pull request?

This PR adds UpdateTable to supported plans in ReplaceNullWithFalseInPredicate.

Why are the changes needed?

This change allows Spark to optimize update conditions like we optimize filters.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

This PR extends the existing test cases to also cover UpdateTable.

@github-actions github-actions bot added the SQL label Dec 15, 2020
@aokolnychyi
Copy link
Contributor Author

@dongjoon-hyun
Copy link
Member

cc @rdblue

@dbtsai
Copy link
Member

dbtsai commented Dec 15, 2020

LGTM. Pending tests.

@SparkQA
Copy link

SparkQA commented Dec 15, 2020

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37445/

@SparkQA
Copy link

SparkQA commented Dec 15, 2020

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37445/

@dongjoon-hyun
Copy link
Member

Merged to master for Apache Spark 3.2.0. Thanks, @aokolnychyi , @dbtsai , @viirya , @rdblue .

@SparkQA
Copy link

SparkQA commented Dec 15, 2020

Test build #132842 has finished for PR 30787 at commit 5a8dffb.

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

@HyukjinKwon
Copy link
Member

LGTM, thanks @aokolnychyi.

@aokolnychyi
Copy link
Contributor Author

Thanks, everyone!

flyrain pushed a commit to flyrain/spark that referenced this pull request Sep 21, 2021
### What changes were proposed in this pull request?

This PR adds `UpdateTable` to supported plans in `ReplaceNullWithFalseInPredicate`.

### Why are the changes needed?

This change allows Spark to optimize update conditions like we optimize filters.

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

This PR extends the existing test cases to also cover `UpdateTable`.

Closes apache#30787 from aokolnychyi/spark-33735.

Authored-by: Anton Okolnychyi <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
(cherry picked from commit 4d56d43)
Signed-off-by: Dongjoon Hyun <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants