LiaCastaneda commented on PR #18921: URL: https://github.com/apache/datafusion/pull/18921#issuecomment-4236543783
Hi @rluvaton @comphead @alamb -- I think this PR is in good shape (cc @gstvg, please correct me if I missed anything or if there are any blockers I’m not aware of). Some of us have already taken a look. There are still some follow-ups tracked in this epic: https://github.com/apache/datafusion/issues/21172, and @gstvg has draft PRs for several of them. Since this is implemented as a separate trait and doesn’t touch existing paths, it seems relatively safe to merge. Would anyone have time to take another look or share thoughts on how we should proceed? I believe there’s quite a bit of interest in this feature, so it would be great to get some guidance on how to proceed. -- 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]
