Skip to content

Conversation

@houqp
Copy link
Member

@houqp houqp commented Jul 5, 2021

Rationale for this change

Follow up for #605. Please review 605 first.

What changes are included in this PR?

Duduplicate using join columns in wildcard projection.

Are there any user-facing changes?

Using join wildcard projection now only projects a single join column from one side of the join clause.

@houqp houqp force-pushed the qp_using_wildcard branch from 6911435 to 68d8efc Compare July 10, 2021 23:51
@houqp houqp marked this pull request as ready for review July 10, 2021 23:54
Copy link
Contributor

@alamb alamb left a comment

Choose a reason for hiding this comment

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

Looks good (and well tested) to me. Thanks @houqp !

@alamb
Copy link
Contributor

alamb commented Jul 13, 2021

I think it is looking great -- thanks @houqp

@alamb alamb merged commit 75a376f into apache:master Jul 13, 2021
@houqp houqp deleted the qp_using_wildcard branch July 13, 2021 19:04
@houqp houqp added bug Something isn't working enhancement New feature or request labels Jul 29, 2021
unkloud pushed a commit to unkloud/datafusion that referenced this pull request Mar 23, 2025
## Which issue does this PR close?
Part of apache#372  and apache#551 

## Rationale for this change
With Spark 4.0, the `SubquerySuite` in Spark fails as Comet scan did not support the scala subquery feature.

## What changes are included in this PR?
Adds the support for scalar subquery pushdown into Comet scan 

## How are these changes tested?
Existing Spark/sql unit tests in `SubquerySuite`
H0TB0X420 pushed a commit to H0TB0X420/datafusion that referenced this pull request Oct 7, 2025
test_array_functions now has 56 passing test cases and 1 expected failure (`array_slice` being the expected failure Ref apache#670).

test_array_function_flatten was broken out as a single test because it was an outlier in terms of test-input.

test_array_function_obj_tests had a different set of asserts, so was broken out for 5 test cases.

Ref apache#671
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants