Skip to content

[1.2] Migrate from Buzz to Guzzle #81

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

Merged
merged 3 commits into from
Nov 2, 2013
Merged

[1.2] Migrate from Buzz to Guzzle #81

merged 3 commits into from
Nov 2, 2013

Conversation

cursedcoder
Copy link
Contributor

From #30

The improvement is pretty good : when running multiple request in the same PHP script, 
Guzzle use CurlMulti as base request handler, using HTTP 1.1 at its best 
(all requests are done inside the same connection) whereas Buzz does 
not support it (it opens a new connection on every request).

Here are the results of my mini benchmark for 50 requests :

Buzz ~ 1min
Guzzle ~ 15seconds

@pilot
Copy link
Contributor

pilot commented Nov 1, 2013

good to merge

cursedcoder added a commit that referenced this pull request Nov 2, 2013
[1.2] Migrate from Buzz to Guzzle
@cursedcoder cursedcoder merged commit e0e6aae into master Nov 2, 2013
@cursedcoder cursedcoder deleted the feature/guzzle branch November 2, 2013 21:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants