ref

Author: geooo109

sqlglot/generators/snowflake.py

@@ -708,21 +708,23 @@ def tonumber_sql(self, expression: exp.ToNumber) -> str:
         precision = expression.args.get("precision")
         scale = expression.args.get("scale")
 
-        is_default = (
-            isinstance(precision, exp.Literal)
-            and precision.name == "38"
-            and isinstance(scale, exp.Literal)
-            and scale.name == "0"
-        )
+        default_precision = isinstance(precision, exp.Literal) and precision.name == "38"
+        default_scale = isinstance(scale, exp.Literal) and scale.name == "0"
+
+        if default_precision and default_scale:
+            precision = None
+            scale = None
+        elif default_scale:
+            scale = None
 
         func_name = "TRY_TO_NUMBER" if expression.args.get("safe") else "TO_NUMBER"
 
         return self.func(
             func_name,
             expression.this,
             expression.args.get("format"),
-            None if is_default else precision,
-            None if is_default else scale,
+            precision,
+            scale,
         )
 
     def timestampfromparts_sql(self, expression: exp.TimestampFromParts) -> str:

tests/dialects/test_snowflake.py

@@ -499,9 +499,8 @@ def test_snowflake(self):
         self.validate_identity(
             "TO_DECIMAL(expr, fmt, precision, scale)", "TO_NUMBER(expr, fmt, precision, scale)"
         )
-        self.validate_identity("TO_NUMBER(expr)")
-        self.validate_identity("TO_NUMBER(expr, fmt)")
-        self.validate_identity("TO_NUMBER(expr, fmt, precision, scale)")
+        self.validate_identity("TO_NUMBER(expr, 38, 0)", "TO_NUMBER(expr)")
+        self.validate_identity("TO_NUMBER(expr, 38)", "TO_NUMBER(expr)")
 
         ast = self.validate_identity("TO_NUMBER('12.3456')")
         self.assertIsInstance(ast, exp.ToNumber)
@@ -527,7 +526,7 @@ def test_snowflake(self):
         self.assertEqual(ast.args.get("precision").name, "10")
         self.assertEqual(ast.args.get("scale").name, "1")
 
-        ast = self.validate_identity("TO_NUMBER('12.3456', 3)", "TO_NUMBER('12.3456', 3, 0)")
+        ast = self.validate_identity("TO_NUMBER('12.3456', 3)")
         self.assertIsInstance(ast, exp.ToNumber)
         self.assertIsNone(ast.args.get("format"))
         self.assertEqual(ast.args.get("precision").name, "3")
@@ -573,9 +572,8 @@ def test_snowflake(self):
         self.validate_identity("TRY_TO_FILE(object_col)")
         self.validate_identity("TRY_TO_FILE('file.csv')")
         self.validate_identity("TRY_TO_FILE('file.csv', 'relativepath/')")
-        self.validate_identity("TRY_TO_NUMBER('123.45')")
-        self.validate_identity("TRY_TO_NUMBER('123.45', '999.99')")
-        self.validate_identity("TRY_TO_NUMBER('123.45', '999.99', 10, 2)")
+        self.validate_identity("TRY_TO_NUMBER(expr, 38, 0)", "TRY_TO_NUMBER(expr)")
+        self.validate_identity("TRY_TO_NUMBER(expr, 38)", "TRY_TO_NUMBER(expr)")
 
         ast = self.validate_identity("TRY_TO_NUMBER('12.3456')")
         self.assertIsInstance(ast, exp.ToNumber)
@@ -605,9 +603,7 @@ def test_snowflake(self):
         self.assertEqual(ast.args.get("scale").name, "1")
         self.assertTrue(ast.args.get("safe"))
 
-        ast = self.validate_identity(
-            "TRY_TO_NUMBER('12.3456', 3)", "TRY_TO_NUMBER('12.3456', 3, 0)"
-        )
+        ast = self.validate_identity("TRY_TO_NUMBER('12.3456', 3)")
         self.assertIsInstance(ast, exp.ToNumber)
         self.assertIsNone(ast.args.get("format"))
         self.assertEqual(ast.args.get("precision").name, "3")