Raphtory icon indicating copy to clipboard operation
Raphtory copied to clipboard

Use references to VertexStore instead of VID to avoid grabbing locks multiple times

Open ljeub-pometry opened this issue 2 years ago • 1 comments

All the vertex-related methods in GraphOps and friends should take a reference to the underlying VertexStore as input so we can pass in locked data when looping over all vertices

ljeub-pometry avatar Aug 31 '23 09:08 ljeub-pometry

This is still not really solved. There is an alternative possibility of directly adding a reference to the storage to a new GraphView struct and to the NodeView and EdgeView so we can handle the locking more easily, something like

pub struct GraphView<'graph, G: GraphViewOps<'graph>> {
    storage: GraphStorage
    view: G
}

ljeub-pometry avatar Dec 10 '24 09:12 ljeub-pometry