You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@datafusion.apache.org by "adriangb (via GitHub)" <gi...@apache.org> on 2026/07/12 18:08:28 UTC
[I] Proto: migrate leaf plans EmptyExec + PlaceholderRowExec [datafusion]
adriangb opened a new issue, #23501:
URL: https://github.com/apache/datafusion/issues/23501
Part of #23494.
`empty.rs`, `placeholder_row.rs`. Leaves — schema only (Schema conversion comes from `datafusion-proto-common`).
---
Follow the pattern in #23494 (reference impl: #23495 / `ProjectionExec`; full end-state draft: #23496). Add `try_to_proto` inside `impl ExecutionPlan for X`, add the inherent `X::try_from_proto`, repoint the decode arm, and **delete the old central arm in the same PR** (that is the proof the hook is reached). Keep the wire format byte-for-byte identical.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org
Re: [I] Proto: migrate leaf plans EmptyExec + PlaceholderRowExec [datafusion]
Posted by "adriangb (via GitHub)" <gi...@apache.org>.
adriangb closed issue #23501: Proto: migrate leaf plans EmptyExec + PlaceholderRowExec
URL: https://github.com/apache/datafusion/issues/23501
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org
Re: [I] Proto: migrate leaf plans EmptyExec + PlaceholderRowExec [datafusion]
Posted by "adriangb (via GitHub)" <gi...@apache.org>.
adriangb commented on issue #23501:
URL: https://github.com/apache/datafusion/issues/23501#issuecomment-5028183746
@847850277 are you going to be picking this up? @andygrove mentioned they would be willing to help so maybe they can review
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org
Re: [I] Proto: migrate leaf plans EmptyExec + PlaceholderRowExec [datafusion]
Posted by "847850277 (via GitHub)" <gi...@apache.org>.
847850277 commented on issue #23501:
URL: https://github.com/apache/datafusion/issues/23501#issuecomment-5029485618
> [@847850277](https://github.com/847850277) are you going to be picking this up? [@andygrove](https://github.com/andygrove) mentioned they would be willing to help so maybe they can review
yes i‘m still working，i will pr this week. Thanks!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org
Re: [I] Proto: migrate leaf plans EmptyExec + PlaceholderRowExec [datafusion]
Posted by "847850277 (via GitHub)" <gi...@apache.org>.
847850277 commented on issue #23501:
URL: https://github.com/apache/datafusion/issues/23501#issuecomment-4953809062
take
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org