Code review comment for lp:~michihenning/thumbnailer/config-access
Revision history for this message
| Michi Henning (michihenning) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
35a32c5
(Get the code!)
Thanks for that!
It's also broken in another way: the number of CPUs really is irrelevant here. I wasn't thinking straight. What we want is 10 if max-downloads is set to 0, and whatever max-downloads is set to, otherwise, regardless of the number of CPUs. (As is, we are too low for a 2-core or 4-core machine because we need max-downloads at 10, even when there are fewer than 8 cores.)
I'll fix it.