Skip to content

fix: Managed writer log message, timeout->flush#536

Merged
kodiakhq[bot] merged 3 commits intocloudquery:mainfrom
erezrokah:fix/log_message
Jan 1, 2023
Merged

fix: Managed writer log message, timeout->flush#536
kodiakhq[bot] merged 3 commits intocloudquery:mainfrom
erezrokah:fix/log_message

Conversation

@erezrokah
Copy link
Copy Markdown
Member

Summary

This fixes the log message to reflect the correct case


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 ✅

@github-actions
Copy link
Copy Markdown

⏱️ Benchmark results

Comparing with 5ffad03

  • DefaultConcurrency-2 resources/s: 11,971 ⬆️ 1.63% increase vs. 5ffad03
  • Glob-2 ns/op: 144.8 ⬇️ 3.31% decrease vs. 5ffad03
  • TablesWithChildrenDefaultConcurrency-2 resources/s: 31,230 ⬆️ 1.73% increase vs. 5ffad03
  • BufferedScanner-2 ns/op: 9.391 ⬆️ 0.44% increase vs. 5ffad03
  • LogReader-2 ns/op: 30.53 ⬆️ 0.03% increase vs. 5ffad03

@kodiakhq kodiakhq bot merged commit 6b0c711 into cloudquery:main Jan 1, 2023
kodiakhq bot pushed a commit that referenced this pull request Jan 1, 2023
🤖 I have created a release *beep* *boop*
---


## [1.16.1](v1.16.0...v1.16.1) (2023-01-01)


### Bug Fixes

* **deps:** Update golang.org/x/exp digest to 738e83a ([#546](#546)) ([bdf3ff1](bdf3ff1))
* **deps:** Update google.golang.org/genproto digest to f9683d7 ([#552](#552)) ([763d22b](763d22b))
* **deps:** Update module github.com/getsentry/sentry-go to v0.16.0 ([#549](#549)) ([b4a0efc](b4a0efc))
* **deps:** Update module github.com/inconshreveable/mousetrap to v1.1.0 ([#555](#555)) ([f449234](f449234))
* **deps:** Update module github.com/mattn/go-isatty to v0.0.17 ([#553](#553)) ([826006f](826006f))
* **deps:** Update module github.com/schollz/progressbar/v3 to v3.12.2 ([#547](#547)) ([b6640b8](b6640b8))
* **deps:** Update module github.com/thoas/go-funk to v0.9.3 ([#548](#548)) ([6e5469a](6e5469a))
* **deps:** Update module golang.org/x/net to v0.4.0 ([#550](#550)) ([9ced5ec](9ced5ec))
* **deps:** Update module golang.org/x/text to v0.5.0 ([#551](#551)) ([1353026](1353026))
* Managed writer log message, timeout->flush ([#536](#536)) ([6b0c711](6b0c711))

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

3 participants