Skip to content

[branch-53] ser/de fetch in FilterExec (#20738) - #20883

Merged
alamb merged 1 commit into
apache:branch-53from
alamb:alamb/backport_20738_branch53
Mar 12, 2026
Merged

[branch-53] ser/de fetch in FilterExec (#20738)#20883
alamb merged 1 commit into
apache:branch-53from
alamb:alamb/backport_20738_branch53

Conversation

@alamb

@alambalamb commented Mar 11, 2026•
edited
Loading

Copy link
Copy Markdown
Contributor

@haohuaijin@alamb
ser/de fetch in FilterExec (apache#20738)
1c3a414
## Which issue does this PR close?
<!--
We generally require a GitHub issue to be filed for all bug fixes and
enhancements and this helps us generate change logs for our releases.
You can link an issue to this PR using the GitHub syntax. For example
`Closesapache#123` indicates that this PR will close issue apache#123.
-->
- Closesapache#20737
## Rationale for this change
FilterExec have fetch filed but not impl the ser/de in proto
## What changes are included in this PR?
add ser/de for fetch in FilterExec
## Are these changes tested?
add one test case
## Are there any user-facing changes?
<!--
If there are user-facing changes then we may require documentation to be
updated before approving the PR.
-->
<!--
If there are any breaking changes to public APIs, please add the `api
change` label.
-->
@github-actionsgithub-actionsBot added proto Related to proto crate physical-plan Changes to the physical-plan crate labels Mar 11, 2026
@alambalamb changed the title ser/de fetch in FilterExec (#20738)[branch-53] ser/de fetch in FilterExec (#20738)Mar 11, 2026
comphead
comphead approved these changes Mar 11, 2026
View reviewed changes

@compheadcomphead 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.

Thanks @alamb

haohuaijin
haohuaijin approved these changes Mar 12, 2026
View reviewed changes

@haohuaijinhaohuaijin 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 @alamb

@alamb
alamb merged commit 2c0a38b into apache:branch-53Mar 12, 2026
55 of 56 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@compheadcompheadcomphead approved these changes
+1 more reviewer
@haohuaijinhaohuaijinhaohuaijin approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

physical-planChanges to the physical-plan crateprotoRelated to proto crate

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@alamb@comphead@haohuaijin