Skip to content

feat: Use write api for automatic data uploads #2338

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
TrevorBergeron wants to merge 7 commits into
base: main
Choose a base branch
from
Open

feat: Use write api for automatic data uploads #2338

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
7f6481c
feat: Use write api for automatic data uploads
TrevorBergeron Dec 17, 2025
177448b
Merge remote-tracking branch 'github/main' into executor_write_api
TrevorBergeron Dec 19, 2025
40323d8
use write api as default for all uploads
TrevorBergeron Dec 19, 2025
8c3c149
Merge remote-tracking branch 'github/main' into executor_write_api
TrevorBergeron Jan 12, 2026
599c404
add config option for default write engine
TrevorBergeron Jan 12, 2026
e867248
fix config app and status bar test
TrevorBergeron Jan 14, 2026
65eb8b3
Merge remote-tracking branch 'github/main' into executor_write_api
TrevorBergeron Jan 14, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 2 additions & 3 deletions bigframes/session/__init__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -976,8 +976,7 @@ def read_pandas(
quota and your data cannot be embedded in SQL due to size or
data type limitations.
* "bigquery_write":
[Preview] Use the BigQuery Storage Write API. This feature
is in public preview.
Use the BigQuery Storage Write API.
Returns:
An equivalent bigframes.pandas.(DataFrame/Series/Index) object

Expand Down Expand Up @@ -1026,7 +1025,7 @@ def _read_pandas(
mem_usage = pandas_dataframe.memory_usage(deep=True).sum()
if write_engine == "default":
write_engine = (
"bigquery_load"
Copy link
Collaborator

@tswast tswast Jan 2, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm a bit worried about https://docs.cloud.google.com/bigquery/docs/sandbox users. Maybe we could provide a global option for the default so that such users can switch back to load jobs (accepting the limitations for data types).

Copy link
Contributor Author

@TrevorBergeron TrevorBergeron Jan 12, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

added config option in new version

"bigquery_write"
if mem_usage > bigframes.constants.MAX_INLINE_BYTES
else "bigquery_inline"
)
Expand Down
2 changes: 1 addition & 1 deletion bigframes/session/bq_caching_executor.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -594,7 +594,7 @@ def _upload_local_data(self, local_table: local_data.ManagedArrowTable):
# Might be better as a queue and a worker thread
with self._upload_lock:
if local_table not in self.cache._uploaded_local_data:
uploaded = self.loader.load_data(
uploaded = self.loader.write_data(
local_table, bigframes.core.guid.generate_guid()
)
self.cache.cache_remote_replacement(local_table, uploaded)
Expand Down