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

Use millisecond granularity for publisher ramp-up period #401

Merged
merged 1 commit into from
Oct 5, 2022

Conversation

acogoluegnes
Copy link
Collaborator

It was seconds, which works for very long ramp-up period (e.g. 60 seconds or more) and a small number of publishers (few 100s), but not short ramp-up period and large numbers of publishers (start 5000 publishers in 15 seconds). In such cases, the workload was not random enough like in real-life and metrics like publisher confirm latency could be much higher than expected.

References #394

It was seconds, which works for very long ramp-up period
(e.g. 60 seconds or more) and a small number of publishers
(few 100s), but not short ramp-up period and large numbers
of publishers (start 5000 publishers in 15 seconds). In such
cases, the workload was not random enough like in real-life
and metrics like publisher confirm latency could be much
higher than expected.

References #394
@acogoluegnes acogoluegnes added this to the 2.19.0 milestone Oct 5, 2022
@acogoluegnes acogoluegnes merged commit c56dabf into main Oct 5, 2022
@acogoluegnes acogoluegnes deleted the millisecond-for-ramp-up-period branch October 5, 2022 07:32
pivotal-rabbitmq-ci pushed a commit that referenced this pull request Oct 5, 2022
Use millisecond granularity for publisher ramp-up period
acogoluegnes added a commit that referenced this pull request Oct 6, 2022
Use a default 1 second random start delay when producer rate instructions
are specified. Use the one specified by the user (including 0, meaning
no delay) otherwise.

References #401
acogoluegnes added a commit that referenced this pull request Oct 7, 2022
Global metrics (shown in the summary at the end of a run) can be
skewed if producers/consumers are started with a delay. Producers
and consumers now notify a listener that resets the global metrics
once everyone is started.

References #401, #406
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant