jackwener commented on PR #5514: URL: https://github.com/apache/arrow-datafusion/pull/5514#issuecomment-1462199484
> > Make great sense to me, thanks @waynexia . > > Can you add a test to cover this case? thanks again. > > Appreciate your suggestion! I add a UT for this change, but only covers the `ExtensionPlan` branch 🥲 I think we can add an extra integration-test to cover this case, look like unit test isn't enough. -- 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]
