Review Request: Make the maximum # of connection per host in KIO::Scheduler user-configurable

[Commit Hook]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104120/#review11142
-----------------------------------------------------------


This review has been submitted with commit 7a5ecbd5316b880f86a2088f420057cdb8d4e2ba by Dawit Alemayehu to branch KDE/4.8.

- Commit Hook


On March 5, 2012, 3:46 p.m., Dawit Alemayehu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104120/
> -----------------------------------------------------------
> 
> (Updated March 5, 2012, 3:46 p.m.)
> 
> 
> Review request for kdelibs and Andreas Hartmetz.
> 
> 
> Description
> -------
> 
> This patch is an alternate approach to making the number of ioslaves connections allowed per server configurable by the user. Unlike the patch proposed in bug# 64647, the approach taken in this patch neither requires any new API additions to KIO::Scheduler nor does it limit configurablity per protocol.
> 
> For restricting the number of ioslaves per server/host all a user would have to do is add a "MaxInstancesPerHost" property to the specific protocols configuration file. For example, to limit the number of ftp connections to "ftp://ftp.foo.bar" to 1, one would only have to add the following to $KDEHOME/share/config/kio_ftprc:
> 
> [ftp.foo.bar]
> MaxInstancesPerHost=1
> 
> But that is not all. The user can make that restriction apply globally to all ftp connections by simply moving property to the global section of of the configuration file. That is move it out of the host ("[ftp.foo.bar]") section. And to make it apply to every protocol, simply move the property to the generic ioslave configuration file, $KDEHOME/share/config/kioslaverc.
> 
> 
> This addresses bug 64647.
>     http://bugs.kde.org/show_bug.cgi?id=64647
> 
> 
> Diffs
> -----
> 
>   kio/kio/scheduler.cpp 6b4ed52 
> 
> Diff: http://git.reviewboard.kde.org/r/104120/diff/
> 
> 
> Testing
> -------
> 
> To verify correct behavior, I added the following settings to kio_ftprc and attempted to download multiple files at the same time. KIO::Scheduler correctly processed the request one at a time:
> 
> [ftp.kde.org]
> MaxConnections=1
> 
> 
> Thanks,
> 
> Dawit Alemayehu
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20120305/841b1896/attachment.htm>