chore: rename feature name from geo to postgis

Author: sunng87

arrow-pg/Cargo.toml

@@ -13,10 +13,10 @@ readme = "../README.md"
 rust-version.workspace = true
 
 [features]
-default = ["arrow", "geo"]
+default = ["arrow", "postgis"]
 arrow = ["dep:arrow"]
 datafusion = ["dep:datafusion"]
-geo = ["postgres-types/with-geo-types-0_7", "dep:geoarrow", "dep:geoarrow-schema", "dep:postgis", "dep:geo-postgis", "pgwire/pg-type-postgis", "dep:geo-traits"]
+postgis = ["postgres-types/with-geo-types-0_7", "dep:geoarrow", "dep:geoarrow-schema", "dep:postgis", "dep:geo-postgis", "pgwire/pg-type-postgis", "dep:geo-traits"]
 
 [dependencies]
 arrow = { workspace = true, optional = true }

arrow-pg/src/datatypes.rs

@@ -2,7 +2,7 @@ use std::sync::Arc;
 
 #[cfg(not(feature = "datafusion"))]
 use arrow::{datatypes::*, record_batch::RecordBatch};
-#[cfg(feature = "geo")]
+#[cfg(feature = "postgis")]
 use arrow_schema::extension::ExtensionType;
 #[cfg(feature = "datafusion")]
 use datafusion::arrow::{datatypes::*, record_batch::RecordBatch};
@@ -124,8 +124,11 @@ pub fn field_into_pg_type(field: &Arc<Field>) -> PgWireResult<Type> {
     match field.extension_type_name() {
         // As of arrow 56, there are additional extension logical type that is
         // defined using field metadata, for instance, json or geo.
-        #[cfg(feature = "geo")]
-        Some(geoarrow_schema::PointType::NAME) => Ok(Type::POINT),
+        //
+        // TODO: there is no fixed Geometry/Geography type id, here we use text
+        // for placeholder.
+        #[cfg(feature = "postgis")]
+        Some(geoarrow_schema::PointType::NAME) => Ok(Type::TEXT),
 
         _ => into_pg_type(arrow_type),
     }

arrow-pg/src/encoder.rs

@@ -20,7 +20,7 @@ use rust_decimal::Decimal;
 use timezone::Tz;
 
 use crate::error::ToSqlError;
-#[cfg(feature = "geo")]
+#[cfg(feature = "postgis")]
 use crate::geo_encoder::encode_geo;
 use crate::list_encoder::encode_list;
 use crate::struct_encoder::encode_struct;
@@ -292,7 +292,7 @@ pub fn encode_value<T: Encoder>(
 ) -> PgWireResult<()> {
     let arrow_type = arrow_field.data_type();
 
-    #[cfg(feature = "geo")]
+    #[cfg(feature = "postgis")]
     if let Some(geoarrow_type) = geoarrow_schema::GeoArrowType::from_extension_field(arrow_field)
         .map_err(|e| PgWireError::ApiError(Box::new(e)))?
     {

arrow-pg/src/lib.rs

@@ -6,7 +6,7 @@
 pub mod datatypes;
 pub mod encoder;
 mod error;
-#[cfg(feature = "geo")]
+#[cfg(feature = "postgis")]
 pub mod geo_encoder;
 pub mod list_encoder;
 pub mod row_encoder;

datafusion-postgres/Cargo.toml

@@ -37,4 +37,4 @@ env_logger = "0.11"
 
 [features]
 default = []
-postgis = ["geodatafusion", "arrow-pg/geo"]
+postgis = ["geodatafusion", "arrow-pg/postgis"]