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

policer: use 1s as the default replicaton cooldown #2617

Merged
merged 1 commit into from
Oct 9, 2023

Conversation

roman-khimov
Copy link
Member

10 objects per second is OK and seems to be similar to what we have on current public networks.

@roman-khimov roman-khimov added the performance More of something per second label Oct 9, 2023
@roman-khimov roman-khimov added this to the v0.39.0 milestone Oct 9, 2023
@codecov
Copy link

codecov bot commented Oct 9, 2023

Codecov Report

Merging #2617 (21ed48f) into master (1e568b6) will not change coverage.
The diff coverage is 0.00%.

❗ Current head 21ed48f differs from pull request most recent head d2ad04c. Consider uploading reports for the commit d2ad04c to get more accurate results

@@           Coverage Diff           @@
##           master    #2617   +/-   ##
=======================================
  Coverage   29.74%   29.74%           
=======================================
  Files         408      408           
  Lines       31215    31215           
=======================================
  Hits         9284     9284           
  Misses      21119    21119           
  Partials      812      812           
Files Coverage Δ
cmd/neofs-node/config/policer/config.go 100.00% <ø> (ø)
pkg/services/policer/policer.go 0.00% <0.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

10 objects per second is OK and seems to be similar to what we have on current
public networks.

Signed-off-by: Roman Khimov <[email protected]>
@roman-khimov roman-khimov merged commit 1006609 into master Oct 9, 2023
7 of 8 checks passed
@roman-khimov roman-khimov deleted the tune-rep-cooldown branch October 9, 2023 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
performance More of something per second
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants