Project

Profile

Help

Task #4114

closed

Rename Remote.connection_limit to Remote.download_concurrency

Added by bmbouter over 5 years ago. Updated over 4 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Category:
-
Sprint/Milestone:
Start date:
Due date:
% Done:

100%

Estimated time:
Platform Release:
Groomed:
Yes
Sprint Candidate:
Yes
Tags:
Sprint:
Sprint 46
Quarter:

Description

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'


Related issues

Related to Pulp - Test #4115: Functionally test downloader concurrency restrictionCLOSED - WONTFIXActions
Actions #1

Updated by bmbouter over 5 years ago

  • Related to Test #4115: Functionally test downloader concurrency restriction added
Actions #2

Updated by dkliban@redhat.com over 5 years ago

  • Groomed changed from No to Yes
Actions #3

Updated by dkliban@redhat.com over 5 years ago

  • Sprint set to Sprint 45
Actions #4

Updated by rchan over 5 years ago

  • Sprint changed from Sprint 45 to Sprint 46
Actions #5

Updated by ppicka over 5 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to ppicka

Added by ppicka over 5 years ago

Revision 9166d985 | View on GitHub

Rename connection_limit to download_concurrency

Rename the 'connection_limit' to 'download_concurrency' throughout the codebase and comments in the codebase to variable name to be more understandable.

re: #4114 https://pulp.plan.io/issues/4114

Required PR: https://github.com/pulp/pulpcore-plugin/pull/23

Signed-off-by: Pavel Picka

Added by ppicka over 5 years ago

Revision 9166d985 | View on GitHub

Rename connection_limit to download_concurrency

Rename the 'connection_limit' to 'download_concurrency' throughout the codebase and comments in the codebase to variable name to be more understandable.

re: #4114 https://pulp.plan.io/issues/4114

Required PR: https://github.com/pulp/pulpcore-plugin/pull/23

Signed-off-by: Pavel Picka

Actions #6

Updated by ppicka over 5 years ago

  • Status changed from ASSIGNED to POST

found only in pulp and pulpcore-plugins from list [devel, pulp, pulp_ansible, pulp_docker, pulp_file, pulp_python, pulp_rpm, pulpcore-plugin]
POST: https://github.com/pulp/pulpcore-plugin/pull/23 & https://github.com/pulp/pulp/pull/3808

Actions #7

Updated by ppicka over 5 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100

Applied in changeset commit:pulpcore-plugin|9105b104ff60dd26183663586990fb421ccc6130.

Actions #8

Updated by daviddavis almost 5 years ago

  • Sprint/Milestone set to 3.0.0
Actions #9

Updated by bmbouter almost 5 years ago

  • Tags deleted (Pulp 3)
Actions #10

Updated by bmbouter over 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF