Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 3 commits into from
Jan 1, 2023

Conversation

erezrokah
Copy link
Contributor

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

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

None yet

3 participants