Pulp

Background¶

The concurrency limiting feature is now present in BaseDownloader which means that all downloaders could support concurrency restriction correctly. The HttpDownloader and FileDownloader both do.

The issue¶

When a setting like Remote.connection_limit is applied to a FileDownloader the name "connection_limit" makes no sense. File I/O has no connections at all, but it does have a notion of concurrency.

Solution¶

Rename the 'connection_limit' to 'download_concurrency' throughout the codebase and comments in the codebase.

@dkliban suggested the name 'download_concurrency'