ubc-systopia / Indaleko

Indaleko Project
GNU Affero General Public License v3.0
0 stars 1 forks source link

Double check for Contains and ContainedBy UUIDs #46

Open hadisinaee opened 7 months ago

hadisinaee commented 7 months ago

When building the validators, I initially used UUIDs in IndalekoRelationshipContains and IndalekoRelationshipContainedBy. However, it turned out that they should be used oppositely to function correctly. For instance, if X and Y represents the UUIDs for Contains and ContainedBy, respectively, we should use Y and X for Contains and ContainedBy.

So, either of these statements should be correct:

  1. Object2 [Contains/ContainedBy] Object1 when we are building IndalekoRelation objects
  2. The UUIDs are used oppositely.

Based on my observations of other ingesters, the first option is more likely.