Skip to content

fix: Batching for mixedbatchwriter - #1374

Merged
kodiakhq[bot] merged 2 commits into
mainfrom
fix-batching-for-mixedbatchwriter
Nov 16, 2023
Merged

kodiakhq[bot] merged 2 commits into
mainfrom
fix-batching-for-mixedbatchwriter

Conversation

@bbernays

Copy link
Copy Markdown
Contributor

Summary


Use the following steps to ensure your PR is ready to be reviewed

  • Read the contribution guidelines 🧑‍🎓
  • Run go fmt to format your code 🖊
  • Lint your changes via golangci-lint run 🚨 (install golangci-lint here)
  • Update or add tests 🧪
  • Ensure the status checks below are successful ✅

Comment thread writers/mixedbatchwriter/mixedbatchwriter.go
@github-actions

github-actions Bot commented Nov 16, 2023

Copy link
Copy Markdown

⏱️ Benchmark results

Comparing with 63fc4bb

  • Glob-8 ns/op: 92.35 ⬆️ 1.02% increase vs. 63fc4bb

@kodiakhq
kodiakhq Bot merged commit ca435cf into main Nov 16, 2023
@kodiakhq
kodiakhq Bot deleted the fix-batching-for-mixedbatchwriter branch November 16, 2023 21:40
kodiakhq Bot pushed a commit that referenced this pull request Nov 16, 2023
🤖 I have created a release *beep* *boop*
---


## [4.18.2](v4.18.1...v4.18.2) (2023-11-16)


### Bug Fixes

* Batching for mixedbatchwriter ([#1374](#1374)) ([ca435cf](ca435cf))
* **deps:** Update module github.com/cloudquery/cloudquery-api-go to v1.4.6 ([#1368](#1368)) ([ea05199](ea05199))
* **deps:** Update module github.com/cloudquery/cloudquery-api-go to v1.5.1 ([#1370](#1370)) ([309b1cb](309b1cb))
* **deps:** Update module github.com/cloudquery/cloudquery-api-go to v1.6.0 ([#1373](#1373)) ([63fc4bb](63fc4bb))
* **deps:** Update module github.com/cloudquery/plugin-pb-go to v1.14.0 ([#1371](#1371)) ([8ec6a34](8ec6a34))

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants