sam-1112 commented on PR #5829: URL: https://github.com/apache/datafusion-comet/pull/5829#issuecomment-5620508967
Thanks for pointing this out. I compared the two PRs, and #5032 already implements the same to_csv routing change, along with broader coverage and the translate change. I missed it because #5578 did not reference #5032 and #5032’s issue-closing field was empty. I’ll close this PR as redundant. After #5032 lands, I can open a small follow-up if any of the additional routing assertions from this PR are still useful. -- 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]
