Skip to content

fix(postgres-spec)!: Move batch_size from the plugin spec to the top level spec#6091

Merged
erezrokah merged 9 commits intocloudquery:mainfrom
erezrokah:fix/postgres_batch_size
Dec 29, 2022
Merged

fix(postgres-spec)!: Move batch_size from the plugin spec to the top level spec#6091
erezrokah merged 9 commits intocloudquery:mainfrom
erezrokah:fix/postgres_batch_size

Conversation

@erezrokah
Copy link
Copy Markdown
Member

@erezrokah erezrokah commented Dec 28, 2022

Summary

Not the same as #6092.
The Postgres plugin doesn't use managed batching so its configuration still works. However since it's available via the top level spec we should use that one for consistency in our docs.

@erezrokah erezrokah requested review from a team and candiduslynx and removed request for a team December 28, 2022 13:19
@erezrokah erezrokah force-pushed the fix/postgres_batch_size branch from d63c5d6 to a3f155b Compare December 28, 2022 16:01
@erezrokah erezrokah added the automerge Automatically merge once required checks pass label Dec 29, 2022
@vercel
Copy link
Copy Markdown

vercel bot commented Dec 29, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
cloudquery-web 🔄 Building (Inspect) Dec 29, 2022 at 9:38AM (UTC)

@erezrokah erezrokah merged commit c504423 into cloudquery:main Dec 29, 2022
@erezrokah erezrokah deleted the fix/postgres_batch_size branch December 29, 2022 09:49
erezrokah pushed a commit that referenced this pull request Dec 29, 2022
🤖 I have created a release *beep* *boop*
---


##
[2.0.0](plugins-destination-postgresql-v1.10.2...plugins-destination-postgresql-v2.0.0)
(2022-12-29)


### ⚠ BREAKING CHANGES

* **postgres-spec:** Move `batch_size` from the plugin spec to the top
level spec
([#6091](#6091))

### Bug Fixes

* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.16.0
([#6098](#6098))
([7bacdf3](7bacdf3))
* **postgres-spec:** Move `batch_size` from the plugin spec to the top
level spec
([#6091](#6091))
([c504423](c504423))

---
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

automerge Automatically merge once required checks pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants