ShayanGho commented on PR #25645: URL: https://github.com/apache/datafusion/pull/25645#issuecomment-5852514256
@namanjain24-sudo Thanks for working on this. The constructor change looks correct, and all 120 FFI unit tests passed locally. #25717 also identifies the delegate-aware child check in pass_runtime_to_children, which remains unchanged here; the new test doesn’t exercise runtime propagation. Since #25749 covers both checks, could we coordinate the overlapping PRs and clarify which one should land? -- 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]
