revert executor change

Author: shuoweil

bigframes/session/bq_caching_executor.py

@@ -196,10 +196,7 @@ def execute(
         self._publisher.publish(bigframes.core.events.ExecutionStarted())
 
         # TODO: Support export jobs in combination with semi executors
-        if (
-            execution_spec.destination_spec is None
-            and execution_spec.promise_under_10gb
-        ):
+        if execution_spec.destination_spec is None:
             plan = self.prepare_plan(array_value.node, target="simplify")
             for exec in self._semi_executors:
                 maybe_result = exec.execute(
@@ -676,14 +673,11 @@ def _execute_plan_gbq(
         )
 
         table_info: Optional[bigquery.Table] = None
-        total_rows = None
         if query_job and query_job.destination:
             table_info = self.bqclient.get_table(query_job.destination)
             size_bytes = table_info.num_bytes
-            total_rows = table_info.num_rows
         else:
             size_bytes = None
-            total_rows = iterator.total_rows
 
         # we could actually cache even when caching is not explicitly requested, but being conservative for now
         if cache_spec is not None:
@@ -710,7 +704,7 @@ def _execute_plan_gbq(
             schema=og_schema,
             query_job=query_job,
             total_bytes=size_bytes,
-            total_rows=total_rows,
+            total_rows=iterator.total_rows,
             total_bytes_processed=iterator.total_bytes_processed,
         )