Refactor!: map duckdb LIST to ArrayAgg (#7552)

Author: georgesittas

sqlglot/generators/snowflake.py

@@ -511,7 +511,6 @@ class SnowflakeGenerator(generator.Generator):
         exp.Levenshtein: unsupported_args("ins_cost", "del_cost", "sub_cost")(
             rename_func("EDITDISTANCE")
         ),
-        exp.List: rename_func("ARRAY_AGG"),
         exp.LocationProperty: lambda self, e: f"LOCATION={self.sql(e, 'this')}",
         exp.LogicalAnd: rename_func("BOOLAND_AGG"),
         exp.LogicalOr: rename_func("BOOLOR_AGG"),

sqlglot/parsers/duckdb.py

@@ -133,6 +133,7 @@ class DuckDBParser(parser.Parser):
         "JSON_ARRAY": lambda args: exp.JSONArray(expressions=args),
         "JSON_EXTRACT_PATH": parser.build_extract_json_with_path(exp.JSONExtract),
         "JSON_EXTRACT_STRING": parser.build_extract_json_with_path(exp.JSONExtractScalar),
+        "LIST": exp.ArrayAgg.from_arg_list,
         "LIST_DISTINCT": exp.ArrayDistinct.from_arg_list,
         "LIST_APPEND": exp.ArrayAppend.from_arg_list,
         "LIST_CONCAT": parser.build_array_concat,

tests/dialects/test_duckdb.py

@@ -693,7 +693,7 @@ def test_duckdb(self):
         self.validate_all(
             "SELECT LIST(col) FROM t",
             write={
-                "duckdb": "SELECT LIST(col) FROM t",
+                "duckdb": "SELECT ARRAY_AGG(col) FROM t",
                 "snowflake": "SELECT ARRAY_AGG(col) FROM t",
             },
         )