alamb commented on PR #15027: URL: https://github.com/apache/datafusion/pull/15027#issuecomment-2702131558
> lgtm thanks @qazxcdswe123 @alamb I'm feeling that having removed the deprecated code and not having migration instructions in `Migration guide` it would be quite a challenge to migrate from version 39 to 46 as we do not keep any traces/hints how API evolved and what is the new API In my opinion any additional small burden related to this API removal (of APIs that have been deprecated for 6 months) is likely tiny compared to all the other major changes. Thus I don't think adding a migration guide adds much value here -- the fact the compiler has been telling people for months it is deprecated is the migration guide -- 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-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
