Fix: ignore/respect nulls generation edge case (#5117)

Author: georgesittas

sqlglot/generator.py

@@ -4287,7 +4287,8 @@ def _embed_ignore_nulls(self, expression: exp.IgnoreNulls | exp.RespectNulls, te
             agg_func = expression.find(exp.AggFunc)
 
             if agg_func:
-                return self.sql(agg_func)[:-1] + f" {text})"
+                agg_func_sql = self.sql(agg_func, comment=False)[:-1] + f" {text})"
+                return self.maybe_comment(agg_func_sql, comments=agg_func.comments)
 
         return f"{self.sql(expression, 'this')} {text}"
 

tests/dialects/test_bigquery.py

@@ -177,6 +177,10 @@ def test_bigquery(self):
         self.validate_identity(
             "CREATE OR REPLACE VIEW test (tenant_id OPTIONS (description='Test description on table creation')) AS SELECT 1 AS tenant_id, 1 AS customer_id",
         )
+        self.validate_identity(
+            "--c\nARRAY_AGG(v IGNORE NULLS)",
+            "ARRAY_AGG(v IGNORE NULLS) /* c */",
+        )
         self.validate_identity(
             'SELECT r"\\t"',
             "SELECT '\\\\t'",