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/05/21 14:17:09 UTC

[I] Port `UnKnownColumn` to use `try_to_proto` / `try_from_proto` [datafusion]

adriangb opened a new issue, #22420:
URL: https://github.com/apache/datafusion/issues/22420

   Sub-issue of #22418. Port `UnKnownColumn` to use the `try_to_proto` / `try_from_proto` hooks added in #21929.
   
   Replace the `UnKnownColumn` arm in `datafusion/proto/src/physical_plan/{to_proto,from_proto}.rs` with `try_to_proto` / `try_from_proto` impls next to `UnKnownColumn`. See the `Column` / `BinaryExpr` migrations in #21929 for reference.


-- 
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] Port `UnKnownColumn` to use `try_to_proto` / `try_from_proto` [datafusion]

Posted by "koopatroopa787 (via GitHub)" <gi...@apache.org>.
koopatroopa787 commented on issue #22420:
URL: https://github.com/apache/datafusion/issues/22420#issuecomment-4520200436

   I’d like to work on this issue.


-- 
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] Port `UnKnownColumn` to use `try_to_proto` / `try_from_proto` [datafusion]

Posted by "adriangb (via GitHub)" <gi...@apache.org>.
adriangb closed issue #22420: Port `UnKnownColumn` to use `try_to_proto` / `try_from_proto`
URL: https://github.com/apache/datafusion/issues/22420


-- 
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] Port `UnKnownColumn` to use `try_to_proto` / `try_from_proto` [datafusion]

Posted by "adriangb (via GitHub)" <gi...@apache.org>.
adriangb commented on issue #22420:
URL: https://github.com/apache/datafusion/issues/22420#issuecomment-4567239740

   Please base your PR on the helpers documented in #22418 (added in #22596). If you've already written this against the older inline pattern, the migration is small — see the second commit of #22596 for examples.
   


-- 
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