19362: Update comment.
authorTom Clegg <tom@curii.com>
Mon, 19 Sep 2022 19:15:25 +0000 (15:15 -0400)
committerTom Clegg <tom@curii.com>
Mon, 19 Sep 2022 19:15:25 +0000 (15:15 -0400)
Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom@curii.com>

sdk/go/arvados/fs_site.go

index eed49296e78c81a9e7d668d58cd51348f738b8d1..531d7968abab360df333619ce5f300f1a809209d 100644 (file)
@@ -158,10 +158,10 @@ func (fs *customFileSystem) mountCollection(parent inode, id string) (inode, err
        if len(id) != 27 {
                // This means id is a PDH, and controller/railsapi
                // returned one of (possibly) many collections with
-               // that PDH. We don't want to expose (e.g., through
-               // Sys()) the metadata from that collection -- only
-               // the fields that are common to all matching
-               // collections, i.e., PDH and manifest.
+               // that PDH. Even if controller returns more fields
+               // besides PDH and manifest text (which are equal for
+               // all matching collections), we don't want to expose
+               // them (e.g., through Sys()).
                coll = Collection{
                        PortableDataHash: coll.PortableDataHash,
                        ManifestText:     coll.ManifestText,