Skip to content

ser/de fetch in FilterExec#20738

Open
haohuaijin wants to merge 3 commits intoapache:mainfrom
haohuaijin:ser/de-fetch
Open

ser/de fetch in FilterExec#20738
haohuaijin wants to merge 3 commits intoapache:mainfrom
haohuaijin:ser/de-fetch

Conversation

@haohuaijin
Copy link
Contributor

@haohuaijin haohuaijin commented Mar 6, 2026

Which issue does this PR close?

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?

@github-actions github-actions bot added proto Related to proto crate physical-plan Changes to the physical-plan crate labels Mar 6, 2026
@haohuaijin haohuaijin closed this Mar 6, 2026
@haohuaijin haohuaijin reopened this Mar 6, 2026
Copy link
Member

@jonahgao jonahgao left a comment

Choose a reason for hiding this comment

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

LGTM

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

Labels

physical-plan Changes to the physical-plan crate proto Related to proto crate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

FilterExec's fetch not impl the ser/de in protobuf

3 participants