polyglot-sql 0.4.2__tar.gz → 0.4.4__tar.gz

{polyglot_sql-0.4.2 → polyglot_sql-0.4.4}/Cargo.lock

@@ -605,7 +605,7 @@ dependencies = [
 
 [[package]]
 name = "polyglot-sql"
-version = "0.4.2"
+version = "0.4.4"
 dependencies = [
  "criterion",
  "once_cell",
@@ -621,7 +621,7 @@ dependencies = [
 
 [[package]]
 name = "polyglot-sql-ffi"
-version = "0.4.2"
+version = "0.4.4"
 dependencies = [
  "cbindgen",
  "polyglot-sql",
@@ -631,11 +631,11 @@ dependencies = [
 
 [[package]]
 name = "polyglot-sql-function-catalogs"
-version = "0.4.2"
+version = "0.4.4"
 
 [[package]]
 name = "polyglot-sql-python"
-version = "0.4.2"
+version = "0.4.4"
 dependencies = [
  "polyglot-sql",
  "pyo3",
@@ -646,7 +646,7 @@ dependencies = [
 
 [[package]]
 name = "polyglot-sql-wasm"
-version = "0.4.2"
+version = "0.4.4"
 dependencies = [
  "console_error_panic_hook",
  "js-sys",

{polyglot_sql-0.4.2 → polyglot_sql-0.4.4}/Cargo.toml

@@ -6,7 +6,7 @@ exclude = [
 ]
 
 [workspace.package]
-version = "0.4.2"
+version = "0.4.4"
 edition = "2021"
 license = "MIT"
 authors = ["polyglot contributors"]

{polyglot_sql-0.4.2 → polyglot_sql-0.4.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: polyglot-sql
-Version: 0.4.2
+Version: 0.4.4
 Classifier: Development Status :: 4 - Beta
 Classifier: Intended Audience :: Developers
 Classifier: License :: OSI Approved :: MIT License

{polyglot_sql-0.4.2 → polyglot_sql-0.4.4}/crates/polyglot-sql/Cargo.toml

@@ -106,7 +106,7 @@ thiserror = { workspace = true }
 unicode-segmentation = { workspace = true }
 stacker = { version = "0.1", optional = true }
 ts-rs = { version = "12.0", features = ["serde-compat"], optional = true }
-polyglot-sql-function-catalogs = { path = "../polyglot-sql-function-catalogs", version = "0.4.2", optional = true, default-features = false }
+polyglot-sql-function-catalogs = { path = "../polyglot-sql-function-catalogs", version = "0.4.4", optional = true, default-features = false }
 
 [dev-dependencies]
 pretty_assertions = "1.4"

{polyglot_sql-0.4.2 → polyglot_sql-0.4.4}/crates/polyglot-sql/src/dialects/fabric.rs

@@ -46,6 +46,8 @@ impl DialectImpl for FabricDialect {
             identifier_quote_style: IdentifierQuoteStyle::BRACKET,
             dialect: Some(DialectType::Fabric),
             null_ordering_supported: false,
+            aggregate_filter_supported: false,
+            cte_recursive_keyword_required: false,
             ..Default::default()
         }
     }
@@ -107,13 +109,7 @@ impl DialectImpl for FabricDialect {
         // Handle DataType::Decimal specially BEFORE T-SQL transform
         // because TSQL converts DECIMAL to NUMERIC, but Fabric wants DECIMAL
         if let Expression::DataType(DataType::Decimal { precision, scale }) = &expr {
-            let name = if let (Some(p), Some(s)) = (precision, scale) {
-                format!("DECIMAL({}, {})", p, s)
-            } else if let Some(p) = precision {
-                format!("DECIMAL({})", p)
-            } else {
-                "DECIMAL".to_string()
-            };
+            let name = Self::decimal_type_name(*precision, *scale);
             return Ok(Expression::DataType(DataType::Custom { name }));
         }
 
@@ -339,21 +335,9 @@ impl FabricDialect {
             },
 
             // DECIMAL -> DECIMAL (override TSQL which converts to NUMERIC)
-            DataType::Decimal { precision, scale } => {
-                if let (Some(p), Some(s)) = (&precision, &scale) {
-                    DataType::Custom {
-                        name: format!("DECIMAL({}, {})", p, s),
-                    }
-                } else if let Some(p) = &precision {
-                    DataType::Custom {
-                        name: format!("DECIMAL({})", p),
-                    }
-                } else {
-                    DataType::Custom {
-                        name: "DECIMAL".to_string(),
-                    }
-                }
-            }
+            DataType::Decimal { precision, scale } => DataType::Custom {
+                name: Self::decimal_type_name(precision, scale),
+            },
 
             // JSON -> VARCHAR
             DataType::Json => DataType::Custom {
@@ -506,21 +490,9 @@ impl FabricDialect {
 
                     // NUMERIC -> DECIMAL (override TSQL's conversion)
                     // Fabric uses DECIMAL, not NUMERIC
-                    "NUMERIC" => {
-                        if let (Some(p), Some(s)) = (precision, scale) {
-                            DataType::Custom {
-                                name: format!("DECIMAL({}, {})", p, s),
-                            }
-                        } else if let Some(p) = precision {
-                            DataType::Custom {
-                                name: format!("DECIMAL({})", p),
-                            }
-                        } else {
-                            DataType::Custom {
-                                name: "DECIMAL".to_string(),
-                            }
-                        }
-                    }
+                    "DECIMAL" | "NUMERIC" => DataType::Custom {
+                        name: Self::decimal_type_name(precision, scale),
+                    },
 
                     // Pass through other custom types unchanged
                     _ => dt,
@@ -542,4 +514,12 @@ impl FabricDialect {
             None => max, // Default to max if not specified
         }
     }
+
+    fn decimal_type_name(precision: Option<u32>, scale: Option<u32>) -> String {
+        match (precision, scale) {
+            (Some(p), Some(s)) => format!("DECIMAL({}, {})", p, s),
+            (Some(p), None) => format!("DECIMAL({})", p),
+            (None, _) => "DECIMAL".to_string(),
+        }
+    }
 }