Page MenuHomeSoftware Heritage
Differential D1698

Add new 'neighbors' API endpoint
ClosedPublic
Actions

Authored by haltode on Jul 8 2019, 10:04 PM.

Details

Reviewers
zack
seirl
Summary

See T1886:

  • Server side: add endpoint handling, main class, unit tests
  • Client side: add endpoint handling
  • Documentation: add endpoint dedicated section

Diff Detail

Repository
rDGRPH Compressed graph representation
Branch
new-neighbors-endpoint
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 6729
Build 9400: arc lint + arc unit

Event Timeline

haltode created this revision.Jul 8 2019, 10:04 PM
Harbormaster completed remote builds in B6729: Diff 5706.Jul 8 2019, 10:04 PM
haltode edited the summary of this revision. (Show Details)Jul 8 2019, 10:09 PM
zack accepted this revision.Jul 9 2019, 8:22 AM
zack added inline comments.
api/server/src/main/java/org/softwareheritage/graph/algo/Neighbors.java
28–33

same critique here of D1698: this is not a great pattern, but doesn't make things worse, so it's OK for now

This revision is now accepted and ready to land.Jul 9 2019, 8:22 AM
haltode closed this revision.Jul 9 2019, 8:46 AM

Closed by commit 27a2871994e9.

haltode mentioned this in T1886: graph API: add endpoint to return the adjacency list of a node.Jul 9 2019, 8:46 AM

Revision Contents
Changeset List

PathSize
api/
client/
swh/
graph/
7 lines
server/
src/
main/
java/
org/
softwareheritage/
graph/
11 lines
algo/
49 lines
test/
java/
org/
softwareheritage/
graph/
122 lines
docs/
29 lines

Diff 5706

View Options

api/client/swh/graph/client.py

Loading...
View Options

api/server/src/main/java/org/softwareheritage/graph/App.java

Loading...
View Options

api/server/src/main/java/org/softwareheritage/graph/algo/Neighbors.java

Loading...
View Options

api/server/src/test/java/org/softwareheritage/graph/NeighborsTest.java

Loading...
View Options

docs/api.rst

Loading...
About · Developer information · Legal