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

Add HTTP request method (GET/POST/PUT, etc) to statistics table #3

Closed
heyman opened this issue Sep 5, 2011 · 6 comments
Closed

Add HTTP request method (GET/POST/PUT, etc) to statistics table #3

heyman opened this issue Sep 5, 2011 · 6 comments

Comments

@heyman
Copy link
Member

heyman commented Sep 5, 2011

No description provided.

@ronniekk
Copy link
Contributor

I had the needs for this a while ago. I ended up with wrapping the client.get/post functions where I prefixed or suffixed the "name" keyword argument with "[GET]" or "[POST]".

A better way would be if Locust handled this internally so that the request method could have its own column in the UI, which could be better designed and sorted on.

@heyman
Copy link
Member Author

heyman commented Jan 11, 2012

I agree. And I think that two different HTTP request methods to the same URL, should then be treated separately in the web UI.

@Jahaja
Copy link
Member

Jahaja commented Jan 16, 2012

I've implemented this functionality in a feature-branch named http-methods (commit 6db337d). Give it a quick try or just check through the commit and see if you see any oddities. The tests should pass but are still a bit WIP regarding using flask rather than gevent for the test web server.

@Jahaja
Copy link
Member

Jahaja commented Feb 22, 2012

This issue were solved in the latest commit.

@Jahaja Jahaja closed this as completed Feb 22, 2012
@heyman
Copy link
Member Author

heyman commented Feb 22, 2012

Nice! Sorry, I was going to check out the branch before, but it totally slipped my mind.

Btw, a nice thing to use if one wants some code review on a feature branch, is to do a pull request from that feature branch into master.

@Jahaja
Copy link
Member

Jahaja commented Feb 22, 2012

Ah, cool, didn't know you could do that.

heyman added a commit that referenced this issue Mar 20, 2012
cpennington added a commit to cpennington/locust that referenced this issue Mar 24, 2015
pancaprima referenced this issue in pancaprima/locust Oct 5, 2017
…er_after_ramp

fixing events remove listener
tyge68 added a commit to tyge68/locust that referenced this issue Sep 15, 2021
# This is the 1st commit message:

locustio#1884 User distribution should happen when new workers comes in

# This is the commit message locustio#2:

locustio#1884 reformatting

# This is the commit message locustio#3:

locustio#1884 Make rebalance interval configurable

# This is the commit message locustio#4:

locustio#1884 Adding integration test for rebalanced

# This is the commit message locustio#5:

locustio#1884 revert change

# This is the commit message locustio#6:

locustio#1886 Simplify solution

# This is the commit message locustio#7:

locustio#1886 adding missing dependency

# This is the commit message locustio#8:

locustio#1884 Adding back enable rebalancing option

# This is the commit message locustio#9:

locustio#1884 adding extra comment the test

# This is the commit message locustio#10:

locustio#1884 renaming as suggested

# This is the commit message locustio#11:

locustio#1884 update help description

# This is the commit message locustio#12:

locustio#1884 handling of missing workers to rebalance
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

3 participants