-
Notifications
You must be signed in to change notification settings - Fork 4
api: add transitive search #66
api: add transitive search #66
Conversation
ead9b84
to
b0a783c
Compare
@classmethod | ||
def equivalents(cls, type_name, value): | ||
"""Get claims with the all the equivalent subjects or objects.""" | ||
all_eq_ids = EquivalentIdentifier.equivalent_ids(type_name, value) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we introduce an eqid
concept, let's change all occurrences of eq_id
to eqid
for greater consistency?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
084b246
to
af8fc12
Compare
@tiborsimko I have applied all te modifications that you have suggested. I have also taken the time to add 2 extra commands:
|
Nice and useful addition with the BTW, since we use
But we can take another issue to think about new nice and clean CLI option tree... |
af8fc12
to
fd28daa
Compare
@tiborsimko I liked your proposal and I have just done it. |
fd28daa
to
dbe9012
Compare
* Adds an extra Model that keeps track of all the equivalent IDs. (closes inveniosoftware#49) * Enables transitive queries to fetch all the related claims (through `/claims?type=xxx&value=yyy&recurse=1`. * Adds a new API resource `/eqids` to list all the equivalent IDs (one can also filter by a specific id using the argument `eqid=`). * Adds command `claimstore index` in order to do operations with indeces. For instance: `claimstore index clear_eqids` and `claimstore index rebuild_eqids`. Signed-off-by: Jose Benito Gonzalez Lopez <jose.benito.gonzalez@cern.ch>
dbe9012
to
ad1195d
Compare
(closes api: optional recursion of transitive relations #49)
/claims?type=xxx&value=yyy&recurse=1
./eqids
to list all the equivalent IDs (one can alsofilter by a specific id using the argument
eqid=
).Signed-off-by: Jose Benito Gonzalez Lopez jose.benito.gonzalez@cern.ch