Skip to content

fix: Fix number of writes reported by postgresql#6757

Merged
kodiakhq[bot] merged 2 commits intomainfrom
postgres-fix-num-writes
Jan 12, 2023
Merged

fix: Fix number of writes reported by postgresql#6757
kodiakhq[bot] merged 2 commits intomainfrom
postgres-fix-num-writes

Conversation

@hermanschaaf
Copy link
Copy Markdown
Contributor

Because we didn't use the batch size, but rather the batch size limit when incrementing the counter for number of records written, the results may have been inaccurate in some cases.

@hermanschaaf hermanschaaf added the automerge Automatically merge once required checks pass label Jan 12, 2023
@kodiakhq kodiakhq bot merged commit 018edf4 into main Jan 12, 2023
@kodiakhq kodiakhq bot deleted the postgres-fix-num-writes branch January 12, 2023 14:15
kodiakhq bot pushed a commit that referenced this pull request Jan 17, 2023
🤖 I have created a release *beep* *boop*
---


## [2.0.6](plugins-destination-postgresql-v2.0.5...plugins-destination-postgresql-v2.0.6) (2023-01-17)


### Bug Fixes

* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.24.2 ([#6695](#6695)) ([694ab9f](694ab9f))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.25.0 ([#6745](#6745)) ([9c41854](9c41854))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.25.1 ([#6805](#6805)) ([9da0ce2](9da0ce2))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.26.0 ([#6839](#6839)) ([6ccda8d](6ccda8d))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.27.0 ([#6856](#6856)) ([545799b](545799b))
* Fix number of writes reported by postgresql ([#6757](#6757)) ([018edf4](018edf4))

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