JuieeAJaviya commented on PR #19632: URL: https://github.com/apache/datafusion/pull/19632#issuecomment-3713511828
Thanks for checking and for pointing this out. You’re right - the current implementation doesn’t compile cleanly across the CI matrix. I also see that there is an existing PR (#18137) addressing this functionality, which I missed earlier. Given the overlap and the CI failures, I’m happy to close this PR and rework my contribution based on existing design patterns or take up a different issue if that’s preferred. Thanks for the guidance. -- 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]
