feat(BigQuery): add StorageWriteApi arrow support#435
Conversation
StorageWriteApi arrow supportStorageWriteApi arrow support
|
Hi @yoshidan, could you please tell me how to run the CI? |
CI triggers are restricted to maintainers, so I have executed it for you. The errors in |
…into arrow-write-api
|
@yoshidan ready for another CI run! |
|
The check-deps is still failing, but since the changes are limited to BigQuery, it should be fine. Please move it out of Draft to "Ready for Review," and I’ll review and merge it. |
|
@yoshidan I can update the hickory dev: |
There was a problem hiding this comment.
Pull request overview
Adds Arrow payload support to the BigQuery Storage Write API path, addressing the requested support for Arrow schemas and record batches alongside existing Protobuf rows.
Changes:
- Adds Arrow-backed
AppendRowsRequestBuilderpayload handling andRecordBatchserialization. - Adds unit coverage for Arrow serialization round-tripping.
- Adds an authenticated storage write test for Arrow rows and updates the Arrow dependency.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
bigquery/src/storage_write/mod.rs |
Adds Arrow payload support and serialization from RecordBatch. |
bigquery/src/storage_write/stream/mod.rs |
Adds a shared Arrow test request helper. |
bigquery/src/storage_write/stream/default.rs |
Adds default stream Arrow write test coverage. |
bigquery/Cargo.toml |
Updates the Arrow dependency version. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
LGTM! |
Hi, I need
StorageWriteApito support Arrow for a project I'm working on.Changes
StorageWriteApi