pepijnve commented on PR #23522: URL: https://github.com/apache/datafusion/pull/23522#issuecomment-4995357669
> This PR is looking really nice to me @pepijnve @adriangb and @jayzhan211 > > However, since the scope has significantly grown since I first reviewed the PR I don't think it would be a good idea to backport this change as is without some "bake time" on main first... > > - https://github.com/apache/datafusion/issues/22547 > > Perhaps we can apply the earlier (smaller) fix for 54? I can go back and redo just the code changes and test additions on a back port branch. Makes sense to keep it as small as possible for a patch. -- 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]
