(sedona-db) branch main updated: fix(rust/sedona-raster-functions): remove documentation field from RS_SetSRID/RS_SetCRS (#637)

[paleolimbot]

This is an automated email from the ASF dual-hosted git repository.

paleolimbot pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/sedona-db.git


The following commit(s) were added to refs/heads/main by this push:
     new b722f156 fix(rust/sedona-raster-functions): remove documentation field 
from RS_SetSRID/RS_SetCRS (#637)
b722f156 is described below

commit b722f1568e25a81286d42cbf8ab63a65a5a4a356
Author: Kristin Cowalcijk <b...@wherobots.com>
AuthorDate: Thu Feb 19 00:47:16 2026 +0800

    fix(rust/sedona-raster-functions): remove documentation field from 
RS_SetSRID/RS_SetCRS (#637)
---
 rust/sedona-raster-functions/src/rs_setsrid.rs | 35 +-------------------------
 1 file changed, 1 insertion(+), 34 deletions(-)

diff --git a/rust/sedona-raster-functions/src/rs_setsrid.rs 
b/rust/sedona-raster-functions/src/rs_setsrid.rs
index a58409d9..2ff6134e 100644
--- a/rust/sedona-raster-functions/src/rs_setsrid.rs
+++ b/rust/sedona-raster-functions/src/rs_setsrid.rs
@@ -23,9 +23,7 @@ use arrow_schema::DataType;
 use datafusion_common::cast::{as_int64_array, as_string_view_array};
 use datafusion_common::error::Result;
 use datafusion_common::{exec_err, DataFusionError, ScalarValue};
-use datafusion_expr::{
-    scalar_doc_sections::DOC_SECTION_OTHER, ColumnarValue, Documentation, 
Volatility,
-};
+use datafusion_expr::{ColumnarValue, Volatility};
 use sedona_expr::scalar_udf::{SedonaScalarKernel, SedonaScalarUDF};
 use sedona_geometry::transform::CrsEngine;
 use sedona_schema::crs::{CachedCrsNormalization, CachedSRIDToCrs};
@@ -45,7 +43,6 @@ pub fn rs_set_srid_with_engine_udf(
         "rs_setsrid",
         vec![Arc::new(RsSetSrid { engine })],
         Volatility::Immutable,
-        Some(rs_set_srid_doc()),
     )
 }
 
@@ -61,7 +58,6 @@ pub fn rs_set_crs_with_engine_udf(
         "rs_setcrs",
         vec![Arc::new(RsSetCrs { engine })],
         Volatility::Immutable,
-        Some(rs_set_crs_doc()),
     )
 }
 
@@ -79,33 +75,6 @@ pub fn rs_set_crs_udf() -> SedonaScalarUDF {
     rs_set_crs_with_engine_udf(None)
 }
 
-fn rs_set_srid_doc() -> Documentation {
-    Documentation::builder(
-        DOC_SECTION_OTHER,
-        "Set the spatial reference system identifier (SRID) of the 
raster".to_string(),
-        "RS_SetSRID(raster: Raster, srid: Integer)".to_string(),
-    )
-    .with_argument("raster", "Raster: Input raster")
-    .with_argument("srid", "Integer: EPSG code to set (e.g., 4326)")
-    .with_sql_example("SELECT RS_SetSRID(RS_Example(), 3857)".to_string())
-    .build()
-}
-
-fn rs_set_crs_doc() -> Documentation {
-    Documentation::builder(
-        DOC_SECTION_OTHER,
-        "Set the coordinate reference system (CRS) of the raster".to_string(),
-        "RS_SetCRS(raster: Raster, crs: String)".to_string(),
-    )
-    .with_argument("raster", "Raster: Input raster")
-    .with_argument(
-        "crs",
-        "String: Coordinate reference system identifier (e.g., 'OGC:CRS84', 
'EPSG:4326')",
-    )
-    .with_sql_example("SELECT RS_SetCRS(RS_Example(), 
'EPSG:3857')".to_string())
-    .build()
-}
-
 // ---------------------------------------------------------------------------
 // RS_SetSRID kernel
 // ---------------------------------------------------------------------------
@@ -407,11 +376,9 @@ mod tests {
     fn udf_metadata() {
         let udf: ScalarUDF = rs_set_srid_udf().into();
         assert_eq!(udf.name(), "rs_setsrid");
-        assert!(udf.documentation().is_some());
 
         let udf: ScalarUDF = rs_set_crs_udf().into();
         assert_eq!(udf.name(), "rs_setcrs");
-        assert!(udf.documentation().is_some());
     }
 
     #[test]