ozankabak commented on PR #14224: URL: https://github.com/apache/datafusion/pull/14224#issuecomment-2633850648
Dear all, this PR is ready for the another round of reviews. I think it is in decent shape, and I'm looking forward to getting more feedback. Old source operators are merely wrappers of the new unified operator, and all tests pass. So we successfully avoided the two-impls issue (thanks @alamb for the suggestion). Without any extra effort, we will already benefit from this architecture by having embedded projection capabilities for all data sources. -- 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]
