Skip to content
This repository has been archived by the owner on Mar 25, 2022. It is now read-only.

some improvements #35

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

some improvements #35

wants to merge 2 commits into from

Conversation

cfz
Copy link

@cfz cfz commented Dec 12, 2018

the first commit add statistics output to influxdb, so we can monitor the stress real time metrics like error rate and tps.
the second commit add a argument validation. we found that if the series number is smaller than the batch size, the current point generation algorithm will generate multiple points with identical timestamp

@cfz cfz changed the title some improvement some improvements Dec 12, 2018
@desa
Copy link
Contributor

desa commented Dec 12, 2018

@cfz good work. Would you be able to sign the CLA

@cfz
Copy link
Author

cfz commented Dec 12, 2018

done :)

@@ -0,0 +1 @@
.idea
Copy link
Contributor

Choose a reason for hiding this comment

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

Was this addition intentional?

Copy link
Author

@cfz cfz Dec 12, 2018

Choose a reason for hiding this comment

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

we use goland as IDE, which by default will generate some metadata in the project path. this one is for excluding those metadata

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