paleolimbot commented on PR #15106: URL: https://github.com/apache/datafusion/pull/15106#issuecomment-2718822202
I'm not sure I'm qualified to review this from an implementation standpoint, but the idea of an extension type registry and a centralized place (here, the LogicalType trait implementation) for behaviour that differs is something I'd like as well, and this PR nicely points out all the places I should be looking 🙂 . -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org