Re: [PR] Refactor substrait producer into multiple files [datafusion]

2025-05-20 Thread via GitHub 


Blizzara commented on PR #16089:
URL: https://github.com/apache/datafusion/pull/16089#issuecomment-2893532984

   Thanks! I took a cursory look as well, looked good 👌 


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


-
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Re: [PR] Refactor substrait producer into multiple files [datafusion]

2025-05-19 Thread via GitHub 


alamb merged PR #16089:
URL: https://github.com/apache/datafusion/pull/16089


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


-
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Re: [PR] Refactor substrait producer into multiple files [datafusion]

2025-05-19 Thread via GitHub 


alamb commented on PR #16089:
URL: https://github.com/apache/datafusion/pull/16089#issuecomment-2891767670

   Since this is just moving code around and is likely uncontroversial I am 
going to merge it in without waiting the customary 24 hours


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


-
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]