paleolimbot commented on code in PR #4423:
URL: https://github.com/apache/datafusion-comet/pull/4423#discussion_r3299743626
##########
docs/geo-functions.md:
##########
@@ -0,0 +1,583 @@
+# Comet Geo Functions
+
+Comet provides 40 geospatial SQL functions registered as Spark SQL extensions.
+All functions execute natively in the Rust/DataFusion engine when Comet is
enabled
+(`spark.comet.exec.enabled=true`). Geometries are represented as WKT strings.
+
+## Constructors
+
+Functions that create geometry values.
+
+### st_geomfromwkt
+
+```sql
+st_geomfromwkt(wkt STRING) -> STRING
+```
+
+Parses a WKT string and returns the geometry. Returns `null` if the input is
`null`.
+
+```sql
+SELECT st_geomfromwkt('POINT(1.0 2.0)');
+-- POINT (1 2)
+```
Review Comment:
A Cargo dependency would be great, but we're a few versions behind Comet and
you're right that they have to align for the non-FFI case. We can figure out a
way to make this work on our end if Comet is interested.
--
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]
