gabotechs commented on PR #20785: URL: https://github.com/apache/datafusion/pull/20785#issuecomment-4031105601
> I would suggest to pin version to 0.62.2 The challenge with pinning the version to a specific patch version is that it might force an unintended dependency change to the people depending on both `datafusion` an `substrait`. I see that in https://github.com/apache/datafusion/pull/20827#discussion_r2911468959 we have not really pinned the version to 0.62.2, the change there is still satisfied by 0.62.3. My impression is that the proper solution is to yank substrait-rs 0.62.3 release, remove it from crates.io, do a major release, as it's a breaking API change. -- 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]
