feat: enable SQLGlot compiler as default

Author: chelsea-lin

bigframes/core/compile/__init__.py

@@ -15,7 +15,7 @@
 
 from bigframes.core.compile.api import test_only_ibis_inferred_schema
 from bigframes.core.compile.configs import CompileRequest, CompileResult
-from bigframes.core.compile.ibis_compiler.ibis_compiler import compile_sql
+from bigframes.core.compile.sqlglot.compiler import compile_sql
 
 __all__ = [
     "test_only_ibis_inferred_schema",

bigframes/core/compile/sqlglot/aggregations/windows.py

@@ -62,10 +62,10 @@ def apply_window_if_present(
 
     # This is the key change. Don't create a spec for the default window frame
     # if there's no ordering. This avoids generating an `ORDER BY NULL` clause.
-    if not window.bounds and not order:
+    if window.is_unbounded and not order:
         return sge.Window(this=value, partition_by=group_by)
 
-    if not window.bounds or not include_framing_clauses:
+    if window.is_unbounded and not include_framing_clauses:
         return sge.Window(this=value, partition_by=group_by, order=order)
 
     kind = (

bigframes/session/direct_gbq_execution.py

@@ -20,7 +20,9 @@
 import google.cloud.bigquery.table as bq_table
 
 from bigframes.core import compile, nodes
-from bigframes.core.compile import sqlglot
+from bigframes.core.compile.ibis_compiler.ibis_compiler import (
+    compile_sql as compile_sql_ibis,
+)
 import bigframes.core.events
 from bigframes.session import executor, semi_executor
 import bigframes.session._io.bigquery as bq_io
@@ -40,7 +42,7 @@ def __init__(
     ):
         self.bqclient = bqclient
         self._compile_fn = (
-            compile.compile_sql if compiler == "ibis" else sqlglot.compile_sql
+            compile_sql_ibis if compiler == "ibis" else compile.compile_sql
         )
         self._publisher = publisher