Skip to content

Backport "Provide field and schema metadata missing on cross joins, and union with null fields" (#12729) - #12974

Merged
alamb merged 2 commits into
apache:branch-42from
matthewmturner:patch/12729
Oct 16, 2024
Merged

Backport "Provide field and schema metadata missing on cross joins, and union with null fields" (#12729)#12974
alamb merged 2 commits into
apache:branch-42from
matthewmturner:patch/12729

Conversation

@matthewmturner

@matthewmturnermatthewmturner commented Oct 16, 2024

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Part of #12813 . Applies fixes from #12729 to DataFusion version 42

Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

@github-actionsgithub-actionsBot added physical-expr Changes to the physical-expr crates core Core DataFusion crate sqllogictest SQL Logic Tests (.slt) labels Oct 16, 2024
@matthewmturner

Copy link
Copy Markdown
ContributorAuthor

@alamb@Xuanwo FYI

@alambalamb changed the title Patch for PR 12729Backport "Provide field and schema metadata missing on cross joins, and union with null fields" (#12729)Oct 16, 2024

@alambalamb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you @matthewmturner -- I updated the title of this PR to be a bit more descriptive when it shows up in the release notes

@alamb

Copy link
Copy Markdown
Contributor

FYI @wiedld

@alamb

alamb commented Oct 16, 2024

Copy link
Copy Markdown
Contributor

For the CI failures, we may need to backport some of this PR as well (for the arrow upgrade): #12724

I will give it a go

@alamb

Copy link
Copy Markdown
Contributor

I backported the fixes for CI: #12977

close/reopen to trigger CI on this PR

@alambalamb closed this Oct 16, 2024
@alambalamb reopened this Oct 16, 2024
@matthewmturner

Copy link
Copy Markdown
ContributorAuthor

i am looking into the CI failure

@matthewmturner

matthewmturner commented Oct 16, 2024

Copy link
Copy Markdown
ContributorAuthor

I see a test passing now (there were none last time) so i think this should be good now 🤞

@alamb
alamb merged commit 81b93e5 into apache:branch-42Oct 16, 2024
@alamb

Copy link
Copy Markdown
Contributor

Thanks agian @matthewmturner

@alamb

Copy link
Copy Markdown
Contributor

(btw I found using git cherry-pick to be a more foolproof way to backport commits -- git often does most of the work for you

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

coreCore DataFusion cratephysical-exprChanges to the physical-expr cratessqllogictestSQL Logic Tests (.slt)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@matthewmturner@alamb