sp-202 commented on PR #4423: URL: https://github.com/apache/datafusion-comet/pull/4423#issuecomment-4556027898
> I created an issue https://github.com/apache/datafusion-comet/issues/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. -- 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]
