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

[Feature] Add -L (limit) on simulate pub #1597

Closed
golfvert opened this issue Mar 3, 2024 · 3 comments
Closed

[Feature] Add -L (limit) on simulate pub #1597

golfvert opened this issue Mar 3, 2024 · 3 comments
Assignees
Labels
CLI MQTTX CLI feature This pr is a feature
Milestone

Comments

@golfvert
Copy link

golfvert commented Mar 3, 2024

Motivation

Now that bench as the option to set a limit (see #1360) I think it would be really good to also have this for the simulate situation.

At the moment:

/app # mqttx simulate -sc weather -c 10 -i 10 -im 500 --limit 100 -t benchmark/test%i
error: unknown option '--limit'

Detailed design

Having the -L with simulate.

Thanks for the great tool !

@golfvert golfvert added the feature This pr is a feature label Mar 3, 2024
@Red-Asuka Red-Asuka self-assigned this Mar 3, 2024
@Red-Asuka Red-Asuka added the CLI MQTTX CLI label Mar 3, 2024
@Red-Asuka
Copy link
Member

Thank you for your valuable suggestion. This only requires some minor adjustments. I will handle it as soon as possible and implement this feature.

@Red-Asuka Red-Asuka changed the title Add -L (limit) on simulate pub [Feature] Add -L (limit) on simulate pub Mar 3, 2024
@ysfscream ysfscream added this to the v1.9.10 milestone Mar 4, 2024
@Red-Asuka
Copy link
Member

@golfvert We have added the -L (limit) option to the simulate command. The new version will be released shortly. Please stay tuned.

@ysfscream
Copy link
Member

Thanks for your feedback. It is now available at: https://github.com/emqx/MQTTX/releases/tag/v1.9.10. Welcome to try it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLI MQTTX CLI feature This pr is a feature
Projects
Status: Done
Development

No branches or pull requests

3 participants