Skip to content

fix(bigquery): BQ batch size#5684

Merged
yevgenypats merged 4 commits intocloudquery:mainfrom
bbernays:bq-batch-size
Dec 16, 2022
Merged

fix(bigquery): BQ batch size#5684
yevgenypats merged 4 commits intocloudquery:mainfrom
bbernays:bq-batch-size

Conversation

@bbernays
Copy link
Copy Markdown
Collaborator

Summary

@bbernays bbernays requested review from a team and shimonp21 and removed request for a team December 15, 2022 21:20
logger zerolog.Logger
spec specs.Destination
metrics plugins.DestinationMetrics
batchSize int
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I see only initialization of this field.
Where is the propagation to the write?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

@yevgenypats yevgenypats merged commit 5288b47 into cloudquery:main Dec 16, 2022
@bbernays bbernays deleted the bq-batch-size branch December 16, 2022 13:05
kodiakhq bot pushed a commit that referenced this pull request Dec 16, 2022
🤖 I have created a release *beep* *boop*
---


## [1.2.0](plugins-destination-bigquery-v1.1.2...plugins-destination-bigquery-v1.2.0) (2022-12-16)


### Features

* **bigquery:** Add configurable batch size ([#5681](#5681)) ([17e110a](17e110a))


### Bug Fixes

* **bigquery:** BQ batch size ([#5684](#5684)) ([5288b47](5288b47))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.12.2 ([#5583](#5583)) ([d721c4e](d721c4e))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.12.3 ([#5639](#5639)) ([6452d0e](6452d0e))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.12.4 ([#5649](#5649)) ([b4aa889](b4aa889))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.12.5 ([#5661](#5661)) ([b354b8a](b354b8a))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants