Project

General

Profile

Actions

Feature #7818

closed

[SDKs] Document that arv-copy accepts collection PDHs (ie content addresses) as arguments

Added by Bryan Cosca over 8 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Normal
Assigned To:
-
Category:
SDKs
Target version:
-
Story points:
0.5
Actions #1

Updated by Brett Smith over 8 years ago

arv-copy does accept collection PDHs. --help could be clearer about this.

Actions #2

Updated by Tom Clegg over 8 years ago

So, something like this in source:sdk/python/arvados/commands/arv_copy.py?
  •      copy_opts.add_argument(
             'object_uuid',
    -        help='The UUID of the object to be copied.')
    +        help='The UUID (or portable data hash) of the object to be copied.')
    
Actions #3

Updated by Bryan Cosca over 8 years ago

My 2 cents: In a collection page, we refer to the pdh as a "Content address", I think we should be consistent here and say "Content address". PDH works for me too, I'm just being nitpicky.

Actions #4

Updated by Brett Smith over 8 years ago

Bryan's comment pokes at a terminology tension here we haven't fully resolved. Most of Arvados refers to this consistently as a "portable data hash." But when we made the relevant Workbench changes, we were very specifically requested to call it "content address."

"Portable data hash" is a term specific to Keep manifests that has a very precise meaning. "Content address" is a general concept that most of our users can grok after just a sentence or two of explanation. A collection's portable data hash serves as a content address for it, but if you try to generate it the same way you generate content addresses for individual Keep data blocks, you'll be disappointed.

I think the number of people who need to know how to generate portable data hashes is comparatively small enough that we should prefer "content address" in all UIs, so I'd vote with Bryan. But maybe we should have that general conversation and at least figure out some basic rules of what term gets used when.

Actions #5

Updated by Brett Smith over 8 years ago

  • Subject changed from arv-copy --by-pdh collection_hash to [SDKs] Document that arv-copy accepts collection PDHs as arguments
  • Category set to SDKs
Actions #6

Updated by Brett Smith over 8 years ago

  • Target version set to Arvados Future Sprints
Actions #7

Updated by Tom Morris over 7 years ago

  • Subject changed from [SDKs] Document that arv-copy accepts collection PDHs as arguments to [SDKs] Document that arv-copy accepts collection PDHs (ie content addresses) as arguments
  • Story points set to 0.5

Does anyone disagree with using "Content Address" here? That term sounds good to me for this type of usage.

Actions #8

Updated by Peter Amstutz over 3 years ago

  • Status changed from New to Closed
Actions #9

Updated by Ward Vandewege over 3 years ago

  • Target version deleted (Arvados Future Sprints)
Actions

Also available in: Atom PDF