Skip to content
This repository was archived by the owner on Jun 20, 2025. It is now read-only.

Calculate scenario dynamic params in settings #66

Merged
merged 12 commits into from
Aug 6, 2018

Conversation

segabriel
Copy link
Member

No description provided.

private boolean consoleReporterEnabled = CONSOLE_REPORTER_ENABLED;
private int users;
Copy link
Contributor

Choose a reason for hiding this comment

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

users -> injectors

private String[] args = new String[] {};
// dynamic params
private Duration rampUpInterval;
private int usersPerRampUpInterval;
Copy link
Contributor

Choose a reason for hiding this comment

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

usersPerRampUpInterval -> injectorsPerRampUpInterval

@@ -264,6 +366,12 @@ private Builder parseArgs() {
String key = keyValue[0];
String value = keyValue[1];
switch (key) {
case "users":
Copy link
Contributor

Choose a reason for hiding this comment

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

"users" -> "injectors"

private String[] args = new String[] {};
// dynamic params
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you plz move this private String[] args = new String[] {}; above line with users/injectors

.rampUpDuration(Duration.ofSeconds(10))
.rampUpInterval(Duration.ofSeconds(1))
.rampUpDuration(Duration.ofSeconds(30))
.users(10_000)
Copy link
Contributor

Choose a reason for hiding this comment

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

.users() -> .injectors()

@artem-v artem-v merged commit 9cb3a63 into develop Aug 6, 2018
@artem-v artem-v deleted the calculate-scenario-dynamic-params-in-settings branch August 6, 2018 14:03
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.

2 participants