Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve performance of apply_update #272

Merged
merged 14 commits into from
Nov 21, 2023
Merged

Conversation

thomaskrause
Copy link
Member

No description provided.

thomaskrause and others added 9 commits November 16, 2023 13:57
This makes use of the newly added `create_empty_corpus` function of the corpus storage, which allows to configure what kind of graph storages the empty corpus should use.
…notationStorage` trait.

This allows to provide optimized implementations for ondisk and inmemory implementations with much better performance than the previous generic one.
As a downside, **this change it breaks the API backward compatibility** because

- the function `get_node_id_from_name()` is removed from the `Graph` struct. Calls to the function can easily be changed to `mygraph.get_node_annos().get_node_ide_from_name()`
- usage of `AnnotationStorage<NodeID>` is replaced with the new trait `NodeAnnotationStorage` (which inherits AnnotationStorage)
- usage of `AnnotationStorage<Edge>` is replaced with the new trait `EdgeAnnotationStorage` (which inherits AnnotationStorage)
The Graph updates are mostly local (e.g. first a `AddNode` and then `AddNodeLabel` events for the same node) and thus the frequency is not important, but that the node has been touched/created recently.
Copy link

codecov bot commented Nov 21, 2023

Welcome to Codecov 🎉

Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.

Thanks for integrating Codecov - We've got you covered ☂️

@thomaskrause thomaskrause merged commit fb77496 into main Nov 21, 2023
13 checks passed
@thomaskrause thomaskrause deleted the feature/apply-update-speed branch November 21, 2023 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant