paleolimbot opened a new issue, #109: URL: https://github.com/apache/sedona-db/issues/109
Currently for s2geography we have two submodules: google/s2geometry and paleolimbot/s2geography. For the release tarball to make sense and include that crate, we need to vendor the source into the release tarball. These are included as submodules because s2geometry in particular has historically had very bad CMake, which is what we are using to build that project. We could (1) use s2geometry from vcpkg/homebrew, which has a few patches applied to potentially make that easier or (2) move away from using a Rust crate and just use the C ABI directly. The second option is more up front work but would simplify the Packaging significantly because it would allow those UDFs to be defined on the s2geography side without Rust in the mix (in our case, it would allow loading them from existing R and Python packages). -- 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]
