paleolimbot commented on issue #4455: URL: https://github.com/apache/datafusion-comet/issues/4455#issuecomment-4557300155
Echoing all Jia's comments plus linking https://github.com/apache/datafusion-comet/pull/4283, which seems like a practical route to adding translations for other functions not limited to just spatial. There are likely some other ways in which adding spatial support as an extension can strengthen extensibility for other cases (this has certainly played out in DuckDB, PostGIS, and DataFusion). While I do think maintaining these externally is a better solution, I'm happy to review anything the Comet community agrees on (i.e., if you'd like to go that direction I'm happy to help get the details right!). -- 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]
