Skip to content

[SPARK-43760][SQL][3.4] Nullability of scalar subquery results#41408

Closed
agubichev wants to merge 3 commits into
apache:branch-3.4from
agubichev:spark-43760-nullability-branch-3.4
Closed

[SPARK-43760][SQL][3.4] Nullability of scalar subquery results#41408
agubichev wants to merge 3 commits into
apache:branch-3.4from
agubichev:spark-43760-nullability-branch-3.4

Conversation

@agubichev
Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Backport of #41287.

Makes sure that the results of scalar subqueries are declared as nullable.

Why are the changes needed?

This is an existing correctness bug, see https://issues.apache.org/jira/browse/SPARK-43760

Does this PR introduce any user-facing change?

Fixes a correctness issue, so it is user-facing.

How was this patch tested?

Query tests.

@github-actions github-actions Bot added the SQL label May 31, 2023
@agubichev
Copy link
Copy Markdown
Contributor Author

@cloud-fan

@HyukjinKwon HyukjinKwon changed the title [SPARK-43760][SQL] Nullability of scalar subquery results [SPARK-43760][SQL][3.4] Nullability of scalar subquery results Jun 1, 2023
@cloud-fan
Copy link
Copy Markdown
Contributor

cloud-fan commented Jun 1, 2023

thanks, merging to 3.4!

cloud-fan pushed a commit that referenced this pull request Jun 1, 2023
### What changes were proposed in this pull request?

Backport of #41287.

Makes sure that the results of scalar subqueries are declared as nullable.

### Why are the changes needed?

This is an existing correctness bug, see https://issues.apache.org/jira/browse/SPARK-43760

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

Fixes a correctness issue, so it is user-facing.

### How was this patch tested?

Query tests.

Closes #41408 from agubichev/spark-43760-nullability-branch-3.4.

Authored-by: Andrey Gubichev <andrey.gubichev@databricks.com>
Signed-off-by: Wenchen Fan <wenchen@databricks.com>
@cloud-fan cloud-fan closed this Jun 1, 2023
@dongjoon-hyun
Copy link
Copy Markdown
Member

Thank you, @agubichev and @cloud-fan !

snmvaughan pushed a commit to snmvaughan/spark that referenced this pull request Jun 20, 2023
### What changes were proposed in this pull request?

Backport of apache#41287.

Makes sure that the results of scalar subqueries are declared as nullable.

### Why are the changes needed?

This is an existing correctness bug, see https://issues.apache.org/jira/browse/SPARK-43760

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

Fixes a correctness issue, so it is user-facing.

### How was this patch tested?

Query tests.

Closes apache#41408 from agubichev/spark-43760-nullability-branch-3.4.

Authored-by: Andrey Gubichev <andrey.gubichev@databricks.com>
Signed-off-by: Wenchen Fan <wenchen@databricks.com>
(cherry picked from commit a2c915d)
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
GladwinLee pushed a commit to lyft/spark that referenced this pull request Oct 10, 2023
### What changes were proposed in this pull request?

Backport of apache#41287.

Makes sure that the results of scalar subqueries are declared as nullable.

### Why are the changes needed?

This is an existing correctness bug, see https://issues.apache.org/jira/browse/SPARK-43760

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

Fixes a correctness issue, so it is user-facing.

### How was this patch tested?

Query tests.

Closes apache#41408 from agubichev/spark-43760-nullability-branch-3.4.

Authored-by: Andrey Gubichev <andrey.gubichev@databricks.com>
Signed-off-by: Wenchen Fan <wenchen@databricks.com>
catalinii pushed a commit to lyft/spark that referenced this pull request Oct 10, 2023
### What changes were proposed in this pull request?

Backport of apache#41287.

Makes sure that the results of scalar subqueries are declared as nullable.

### Why are the changes needed?

This is an existing correctness bug, see https://issues.apache.org/jira/browse/SPARK-43760

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

Fixes a correctness issue, so it is user-facing.

### How was this patch tested?

Query tests.

Closes apache#41408 from agubichev/spark-43760-nullability-branch-3.4.

Authored-by: Andrey Gubichev <andrey.gubichev@databricks.com>
Signed-off-by: Wenchen Fan <wenchen@databricks.com>
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.

3 participants