andygrove commented on PR #4423: URL: https://github.com/apache/datafusion-comet/pull/4423#issuecomment-4556199572
> > I created an issue #4455 so that we can have the discussion about whether Comet should support geo functions, and if so, what are the various options. Let's have this discussion before creating PRs. > > Hey,@andygrove the current implementation on my branch uses WKB as the geometry representation and the pure Rust geo crate for all operations — there is no dependency on the GEOS C library. This addresses both the performance concern (WKB vs WKT) and the build dependency concern raised in the discussion. As I've mentioned a couple of times already, we need to have a discussion on the mailing list about adding geospatial functions to Comet since this would be a significant change to the project scope. It would be great if you could participate in the discussion. A good starting point would be for you to explain the use cases that you are working with that benefit from this work. -- 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]
