Project

Profile

Help

Task #2968

Rename DeferredArtifact and remove DownloadCatalog.

Added by jortel@redhat.com over 3 years ago. Updated about 1 year ago.

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

100%

Estimated time:
Platform Release:
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Sprint 24
Quarter:

Description

I think the name can be improved. The record is always created (even when downloading is not deferred) and represents an artifact in a remote or external repository. It's used by the streamer for downloading and Publishers for information when a published Artifact does not exist. So, having deferred in the name seems inappropriate.

Perhaps something like: RemoteArtifact or going back to something like ArtifactCatalog. Suggestions?

Also, this model takes the place of the DownloadCatalog and it should be deleted.

This affects the plugin API.

Associated revisions

Revision bba14bf2 View on GitHub
Added by daviddavis over 3 years ago

Rename DeferredArtifact and remove DownloadCatalog

fixes #2968 https://pulp.plan.io/issues/2968

Revision bba14bf2 View on GitHub
Added by daviddavis over 3 years ago

Rename DeferredArtifact and remove DownloadCatalog

fixes #2968 https://pulp.plan.io/issues/2968

History

#1 Updated by mhrivnak over 3 years ago

+1 to the uniqueness constraint that's obviously missing.

For the name aspect, I suggest adding to the description that the name will be changed only if a better one can be agreed upon. I wouldn't want to hold up the rest of these improvements if we have trouble settling on a name.

One thing I'm not crazy about with regard to DeferredArtifact is that "defer" implies that retrieval has been delayed but will eventually happen. I think for most on-demand use cases, a lot of the content will never be retrieved, and that's an valued part of the feature.

It's often a good idea to focus on the nature of the data itself, not necessarily how or why it gets used. RemoteArtifact captures that it's a reference to a file outside of Pulp. ExternalArtifact might also work. ForeignArtifact says less about where the artifact is, but just focuses on the fact that it's not here. AvailableArtifact would heavily imply that the artifact is not locally stored, but that the record contains info on how to get it. ArtifactDownload emphasizes that this record is really about downloading, and the download info happens to be for an Artifact. OnDemandArtifact would match other language we use and I think is clear.

Any of that resonating well? Or sparking other ideas?

#2 Updated by jortel@redhat.com over 3 years ago

  • Subject changed from DeferredArtifact improvments. to Rename DeferredArtifact and remove DownloadCatalog.
  • Description updated (diff)

#3 Updated by jortel@redhat.com over 3 years ago

mhrivnak wrote:

+1 to the uniqueness constraint that's obviously missing.

For the name aspect, I suggest adding to the description that the name will be changed only if a better one can be agreed upon. I wouldn't want to hold up the rest of these improvements if we have trouble settling on a name.

Agreed. Let's deal with adding the unique constraint in a separate issue/pr.

#4 Updated by ttereshc over 3 years ago

  • Sprint/Milestone set to 43
  • Triaged changed from No to Yes

We still need to agree on the name.

#5 Updated by jortel@redhat.com over 3 years ago

  • Tracker changed from Issue to Task
  • % Done set to 0

#6 Updated by daviddavis over 3 years ago

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

#7 Updated by daviddavis over 3 years ago

The two names I am partial to are RemoteArtifact and ArtifactDownload. Since there's already been support raised for the former, I'm tempted to use it unless anyone opposes.

#8 Updated by daviddavis over 3 years ago

  • Status changed from ASSIGNED to POST

#9 Updated by daviddavis over 3 years ago

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

#10 Updated by bmbouter about 3 years ago

  • Tags deleted (Pulp 3 Plugin Writer Alpha)

Cleaning up Redmine tags

#11 Updated by bmbouter almost 3 years ago

  • Sprint set to Sprint 24

#12 Updated by bmbouter almost 3 years ago

  • Sprint/Milestone deleted (43)

#13 Updated by daviddavis almost 2 years ago

  • Sprint/Milestone set to 3.0.0

#14 Updated by bmbouter over 1 year ago

  • Tags deleted (Pulp 3)

#15 Updated by bmbouter about 1 year ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF