Page MenuHomeSoftware Heritage

API: add /node and /edge endpoints to check for node/edge existence
Changes PlannedPublic

Authored by zack on Nov 30 2019, 3:39 PM.

Details

Reviewers
seirl
Group Reviewers
Reviewers
Test Plan

tox

Diff Detail

Repository
rDGRPH Graph service
Branch
feature/api-node-edge-methods
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 9547
Build 14055: tox-on-jenkinsJenkins
Build 14054: arc lint + arc unit

Event Timeline

zack created this revision.Nov 30 2019, 3:39 PM
zack updated this revision to Diff 8387.Nov 30 2019, 8:46 PM
  • API doc: untabify
seirl requested changes to this revision.Dec 4 2019, 2:27 PM

I think the API could be more REST:

  • /node/<src> that gives you the list of neighbors of src (and that you can HEAD to know if it's present or not)
  • /edge/<src>/<dst> is already REST

We could merge /neighbors like that.

This revision now requires changes to proceed.Dec 4 2019, 2:27 PM
zack planned changes to this revision.Dec 6 2019, 11:39 AM
zack updated this revision to Diff 8464.Dec 6 2019, 11:41 AM

Updating D2380: API: add /node and /edge endpoints to check for node/edge existence

zack planned changes to this revision.Dec 6 2019, 11:41 AM