influxdata / influx-stress

New tool for generating artificial load on InfluxDB
MIT License
121 stars 43 forks source link

Use pps as the source of truth #21

Closed nathanielc closed 7 years ago

nathanielc commented 7 years ago

I set a pps of 1 to just test I was getting the correct data output and I got a panic. Looking at the code, any time pps < batchSize a panic would occur.

Since this is not a good user experience, either pps needs to be recomputed or batchSize does. I decided that making pps the source of truth for what the user wants makes the most sense, because batch size is about how the data is transferred not how much data is transferred.

This change makes it so that if pps < batchSize the batchSize it changed to be equal to pps and concurrency is set to 1.