-
Notifications
You must be signed in to change notification settings - Fork 163
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: bump DataFusion to rev f4e519f #783
Conversation
@andygrove
|
q97 (partial) select sum(case when ssci.customer_sk is not null and csci.customer_sk is null then 1 else 0 end) store_only
,sum(case when ssci.customer_sk is null and csci.customer_sk is not null then 1 else 0 end) catalog_only
,sum(case when ssci.customer_sk is not null and csci.customer_sk is not null then 1 else 0 end) store_and_catalog
from ssci full outer join csci on (ssci.customer_sk=csci.customer_sk
and ssci.item_sk = csci.item_sk) Seems like this may be a regression in DataFusion. I did make a change to |
The test passes for me locally, but consistently fails in CI |
I can reproduce locally now |
The regression is caused by apache/datafusion#11627 |
Do we just upgrade to the commit before one that causes CI failure? |
Yes, I pushed a commit to just upgrade to the point where min/max API changed |
* chore: bump DataFusion to rev c6f0d3c * fix style * Trigger Build * Update native/Cargo.toml --------- Co-authored-by: Andy Grove <[email protected]>
Which issue does this PR close?
N/A
Rationale for this change
Weekly DataFusion revision bump. Here is the original PR
What changes are included in this PR?
Upgrade to DataFusion revision f4e519f9d , which includes these changes:
How are these changes tested?