Task #2968
closedRename DeferredArtifact and remove DownloadCatalog.
100%
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.
Updated by mhrivnak over 7 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?
Updated by jortel@redhat.com over 7 years ago
- Subject changed from DeferredArtifact improvments. to Rename DeferredArtifact and remove DownloadCatalog.
- Description updated (diff)
Updated by jortel@redhat.com over 7 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.
Updated by ttereshc over 7 years ago
- Sprint/Milestone set to 43
- Triaged changed from No to Yes
We still need to agree on the name.
Updated by jortel@redhat.com over 7 years ago
- Tracker changed from Issue to Task
- % Done set to 0
Updated by daviddavis over 7 years ago
- Status changed from NEW to ASSIGNED
- Assignee set to daviddavis
Updated by daviddavis over 7 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.
Updated by daviddavis over 7 years ago
- Status changed from ASSIGNED to POST
Added by daviddavis over 7 years ago
Added by daviddavis over 7 years ago
Revision bba14bf2 | View on GitHub
Rename DeferredArtifact and remove DownloadCatalog
Updated by daviddavis over 7 years ago
- Status changed from POST to MODIFIED
- % Done changed from 0 to 100
Applied in changeset pulp|bba14bf2d96a8bdaa5137ec5bb7f137dd67e0ca9.
Updated by bmbouter about 7 years ago
- Tags deleted (
Pulp 3 Plugin Writer Alpha)
Cleaning up Redmine tags
Updated by bmbouter about 5 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Rename DeferredArtifact and remove DownloadCatalog
fixes #2968 https://pulp.plan.io/issues/2968