Features/cached graphs

[shivamka1]

What changes were proposed in this pull request?

#1649

Why are the changes needed?

Same as above

Does this PR introduce any user-facing change? If yes is this documented?

No

How was this patch tested?

Uni, integration and manual tests

Issues

If this resolves any issues, please link to them here, the format is a KEYWORD followed by @_
KEYWORDS available are close, closes, closed, fix, fixes, fixed, resolve, resolves, resolved.
Please delete this text before creating your PR

Are there any further changes required?

No

[miratepuffin]

Many random notes - as we discussed I think we need to rethink the different pathways to upload a graph tomorrow and cut some of the confusion out of these functions.

[miratepuffin]

After discussion, we should tidy the APIs to provide the following:

server = RaphtoryServer(working_dir="/graphs",port=1736,config = ...)

server.start()

server.stop()

server.wait()

server.run()



c=server.get_client()

c=RaphtoryClient("localhost:1736")

c.send_graphs(hashmap_of_graphs,overwrite=True,namespace="ben")

c.load_from_path(path_to_folder_of_graphs,overwrite=True,namespace="shivam")

c.query(...)

[shivamka1]

After discussion, we should tidy the APIs to provide the following:

server = RaphtoryServer(working_dir="/graphs",port=1736,config = ...)

server.start()

server.stop()

server.wait()

server.run()



c=server.get_client()

c=RaphtoryClient("localhost:1736")

c.send_graphs(hashmap_of_graphs,overwrite=True,namespace="ben")

c.load_from_path(path_to_folder_of_graphs,overwrite=True,namespace="shivam")

c.query(...)

Implemented this leaving the namespace argument which we will address once load is sorted

[fabianmurariu]

if we absolutely must println, we should use tracing https://docs.rs/tracing/latest/tracing/attr.instrument.html otherwise we should remove them

[miratepuffin]

• Disable load graph from path for both GraphQL and python apis (keep internal function as we may want to add back).
• Change the return type of python mv,copy,delete to just Result<()>
• Decode the graph within python receive (you have the code already in a test)
• Nuke the tracing as we are going to do our own one
• Disable auth for user APIs until we have audited

[miratepuffin]

Can we change this to an Option which would also allow renaming of the graph on copy

[miratepuffin]

Final bits to check through before merge

[miratepuffin]

The final review!