Update packages/bigframes/bigframes/core/compile/sqlglot/expressions/ai_ops.py

sycai · gemini-code-assist[bot] · web-flow · commit 08a8ffa039af · 2026-04-28T15:49:18.000-07:00
Co-authored-by: gemini-code-assist[bot] &lt;176961590+gemini-code-assist[bot]@users.noreply.github.com&gt;
diff --git a/packages/bigframes/bigframes/core/compile/sqlglot/expressions/ai_ops.py b/packages/bigframes/bigframes/core/compile/sqlglot/expressions/ai_ops.py
@@ -139,11 +139,11 @@ def _construct_named_args(op: ops.ScalarOp) -> list[sge.Kwarg]:
                     expression=sge.JSON(this=sge.Literal.string(value)),
                 )
             )
-        elif field == "optimization_mode":
+        elif field == "optimization_mode" and value is not None:
             args.append(
                 sge.Kwarg(this=field, expression=sge.Literal.string(value.upper()))
             )
-        elif field == "max_error_ratio":
+        elif field == "max_error_ratio" and value is not None:
             args.append(sge.Kwarg(this=field, expression=sge.Literal.number(value)))
         elif field == "request_type":
             args.append(

Original file line number	Diff line number	Diff line change
`@@ -139,11 +139,11 @@ def _construct_named_args(op: ops.ScalarOp) -> list[sge.Kwarg]:`
`139`	`139`	`expression=sge.JSON(this=sge.Literal.string(value)),`
`140`	`140`	`)`
`141`	`141`	`)`
`142`		`- elif field == "optimization_mode":`
	`142`	`+ elif field == "optimization_mode" and value is not None:`
`143`	`143`	`args.append(`
`144`	`144`	`sge.Kwarg(this=field, expression=sge.Literal.string(value.upper()))`
`145`	`145`	`)`
`146`		`- elif field == "max_error_ratio":`
	`146`	`+ elif field == "max_error_ratio" and value is not None:`
`147`	`147`	`args.append(sge.Kwarg(this=field, expression=sge.Literal.number(value)))`
`148`	`148`	`elif field == "request_type":`
`149`	`149`	`args.append(`