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

Change CSV column names #1317

Closed
heyman opened this issue Apr 7, 2020 · 2 comments
Closed

Change CSV column names #1317

heyman opened this issue Apr 7, 2020 · 2 comments
Milestone

Comments

@heyman
Copy link
Member

heyman commented Apr 7, 2020

Since we're changing the CSV columns (#1316) we should also consider changing the column names to be more consistent.

Here are the current column names for the history CSV
Timestamp
User count
Type
Name
# requests
# failures
Requests/s
Requests Failed/s
Median response time
Average response time
Min response time
Max response time
Average Content Size
50%
66%
75%
80%
90%
95%
98%
99%
99.9%
99.99%
99.999
100%
Column names for the request stats CSV
Type
Name
# requests
# failures
Median response time
Average response time
Min response time
Max response time
Average Content Size
Requests/s
Requests Failed/s
50%
66%
75%
80%
90%
95%
98%
99%
99.9%
99.99%
99.999
100%
Column names for the failures CSV
Method
Name
Error
Occurrences
Column names for the exceptions CSV
Count
Message
Traceback
Nodes

For the use-case where people use the CSV data programmatically, I guess it would make sense to use more variable like column names such as user_count, request_count, requests_per_second, response_time_99, etc.

However there's also the use-case of simply importing the CSV into a spreadsheet, and it that case more human readable column names makes more sense.

What do you think?

@heyman heyman added this to the 1.0 milestone Apr 7, 2020
@cyberw
Copy link
Collaborator

cyberw commented Apr 9, 2020

I'm definitely ok with changing, but I'm not personally invested in it and dont want it to block 1.0 release :)

@heyman
Copy link
Member Author

heyman commented Apr 9, 2020

I've decided to change the naming to be more consistent, and to append a "Total " prefix to the columns in the history CSV that contains stats for the whole (total) test run. The new column names for the history CSV will be:

Timestamp
User Count
Type
Name
Requests/s
Failures/s
50%
66%
75%
80%
90%
95%
98%
99%
99.9%
99.99%
99.999%
100%
Total Request Count
Total Failure Count
Total Median Response Time
Total Average Response Time
Total Min Response Time
Total Max Response Time
Total Average Content Size

@heyman heyman closed this as completed in 1906dba Apr 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants