revert: ibis change

Author: tswast

bigframes/core/compile/ibis_compiler/ibis_compiler.py

@@ -221,15 +221,18 @@ def _table_to_ibis(
     physical_schema = ibis_bigquery.BigQuerySchema.to_ibis(
         list(source.table.physical_schema)
     )
-    import bigframes.session._io.bigquery
-
-    sql = bigframes.session._io.bigquery.to_query(
-        full_table_name,
-        columns=scan_cols,
-        sql_predicate=source.sql_predicate,
-        time_travel_timestamp=source.at_time,
-    )
-    return ibis_bigquery.Backend().sql(schema=physical_schema, query=sql)
+    if source.at_time is not None or source.sql_predicate is not None:
+        import bigframes.session._io.bigquery
+
+        sql = bigframes.session._io.bigquery.to_query(
+            full_table_name,
+            columns=scan_cols,
+            sql_predicate=source.sql_predicate,
+            time_travel_timestamp=source.at_time,
+        )
+        return ibis_bigquery.Backend().sql(schema=physical_schema, query=sql)
+    else:
+        return ibis_api.table(physical_schema, full_table_name).select(scan_cols)
 
 
 @_compile_node.register

tests/unit/core/test_pyformat.py

@@ -562,6 +562,9 @@ def test_pyformat_with_bigframes_dataframe_biglake_table(session):
     # For BigLake, we now expect a SUBQUERY, not a view reference.
     # The subquery should have correctly quoted 4-part ID.
     assert "SELECT" in got_sql
-    assert "FROM `my-project`.`my-catalog`.`my-namespace`.`my-table`" in got_sql
+    assert project_id in got_sql
+    assert catalog_id in got_sql
+    assert namespace_id in got_sql
+    assert table_id in got_sql
     assert got_sql.startswith("SELECT * FROM (SELECT")
     assert got_sql.endswith(")")