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

Change default max_concurrent_files to 512 #423

Merged
merged 1 commit into from
Sep 14, 2021
Merged

Conversation

djaglowski
Copy link
Member

@djaglowski djaglowski commented Sep 14, 2021

Description of Changes

Please check that the PR fulfills these requirements

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • Add a changelog entry (for non-trivial bug fixes / features)
  • CI passes

@djaglowski
Copy link
Member Author

Log Files Logs / Second CPU Avg (%) CPU Avg Δ (%) Memory Avg (MB) Memory Avg Δ (MB)
1 1000 1.3793215 -0.17242444 129.41339 +2.772766
1 5000 4.3103623 -0.48279476 136.34657 +2.9057007
1 10000 9.913996 +0.20705509 145.1056 +4.483032
1 50000 49.499508 +2.360962 178.32759 +5.888748
1 100000 92.45093 +0.82910156 230.07193 -9.212555
10 100 1.8793254 +0.15511215 132.0718 -2.4773712
10 500 5.6036005 -0.18963814 138.6149 -2.0849915
10 1000 11.0000725 -1.5862617 146.7772 +0.4096222
10 5000 49.638767 -3.4152374 177.1339 -0.6011505
10 10000 101.45197 -2.5137558 233.74892 +13.088364

@jsirianni jsirianni self-requested a review September 14, 2021 19:17
@jsirianni jsirianni merged commit 769c7e0 into master Sep 14, 2021
@jsirianni jsirianni deleted the max-files-512 branch September 14, 2021 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants