coderfender commented on PR #3351: URL: https://github.com/apache/datafusion-comet/pull/3351#issuecomment-3880428192
Great suggestion @comphead . I initially added a custom listerner to check that the native writer is called. However, I figured it might be cleaner to limit the scope of this PR to just tag the failing tests and follow up with more work to build on this. Do you think it is a good idea to add that verification in this PR or would you prefer the listener changes in a different one ? -- 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]
