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

Split providers by smaller groups #45

Merged
merged 1 commit into from
Dec 11, 2014
Merged

Split providers by smaller groups #45

merged 1 commit into from
Dec 11, 2014

Conversation

lavoiesl
Copy link
Member

There are 3 problems with the current separation (the one I did in #29):

  1. The providers get update too frequently.
  2. Old is too big, if your connection is not fast enough, composer will timeout and fail horribly.
  3. There is actually a bug, last-year is not really last year.

Problem 2 is the reason why I’m doing this. It is especially painful if wpackagist’s server gets overloaded.

Before:

+---------------+----------+---------+
| provider      | packages | size    |
+---------------+----------+---------+
| old           | 41388    | 4.5 MiB |
| last-year     | 19803    | 2.2 MiB |
| last-2-months | 9840     | 1.1 MiB |
| last-week     | 1054     | 117 KiB |
+---------------+----------+---------+

After:

+-----------+----------+---------+
| provider  | packages | size    |
+-----------+----------+---------+
| 2013      | 11216    | 1.2 MiB |
| 2012      | 9145     | 1.0 MiB |
| old       | 15046    | 1.6 MiB |
| 2011      | 7317     | 820 KiB |
| 2014-06   | 5355     | 603 KiB |
| 2014-09   | 9241     | 1.0 MiB |
| 2014-03   | 3861     | 433 KiB |
| 2014-12   | 8211     | 927 KiB |
| this-week | 2693     | 303 KiB |
+-----------+----------+---------+

@lavoiesl
Copy link
Member Author

@tamlyn Do you have objections with me merging this ?

@tamlyn
Copy link
Member

tamlyn commented Dec 10, 2014

Looks good. I'll do a deployment when I get a chance but that probably won't be until the week after next.

lavoiesl added a commit that referenced this pull request Dec 11, 2014
Split providers by smaller groups
@lavoiesl lavoiesl merged commit 71ae76d into outlandishideas:master Dec 11, 2014
@lavoiesl lavoiesl deleted the split-files branch December 11, 2014 22:58
@tamlyn
Copy link
Member

tamlyn commented Dec 14, 2014

Deployed.

@lavoiesl
Copy link
Member Author

I’ll have to test more extensively tomorrow while the servers are on higher demand, but so far, the loading is much more smooth and reliable. This is a very welcomed change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants