Skip to content
This repository was archived by the owner on Aug 12, 2022. It is now read-only.

feat: Calculate max goroutines based on file limit#337

Merged
roneli merged 8 commits intocloudquery:mainfrom
roneli:feat-calculate-limits-with-max-open-files
Jun 15, 2022
Merged

feat: Calculate max goroutines based on file limit#337
roneli merged 8 commits intocloudquery:mainfrom
roneli:feat-calculate-limits-with-max-open-files

Conversation

@roneli
Copy link
Copy Markdown
Contributor

@roneli roneli commented Jun 14, 2022

🎉 Thank you for making CloudQuery awesome by submitting a PR 🎉

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 ✅

@roneli roneli marked this pull request as ready for review June 15, 2022 12:09
@roneli roneli requested review from a team, bbernays, erezrokah and shimonp21 and removed request for a team and erezrokah June 15, 2022 12:09
Comment thread helpers/limit/limits.go
Comment thread helpers/limit/limits.go Outdated
}

if mfo < total {
return uint64(float64(mfo) * 0.3)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should the 0.3 also be a const?

@roneli roneli merged commit fb429b8 into cloudquery:main Jun 15, 2022
@roneli roneli deleted the feat-calculate-limits-with-max-open-files branch June 15, 2022 13:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants