Skip to content

Conversation

@ljeub-pometry
Copy link
Collaborator

What changes were proposed in this pull request?

  • support for atomic replacements of graphs/folders/namespaces
  • rework of the internal folder structures

Why are the changes needed?

fixes #2377 (reenables the tests and fixes serialisation)
fixes #2384

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

How was this patch tested?

Are there any further changes required?

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark 'Rust Benchmark'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 2.

Benchmark suite Current: 6bfbced Previous: 102a9fe Ratio
local_triangle_count/local_triangle_count 1603139 ns/iter (± 211662) 706696 ns/iter (± 8298) 2.27
lotr_graph/num_edges 4 ns/iter (± 0) 0 ns/iter (± 0) +∞
lotr_graph/num_nodes 79 ns/iter (± 0) 1 ns/iter (± 0) 79
lotr_graph_materialise/materialize 3057237 ns/iter (± 3536) 1468616 ns/iter (± 9419) 2.08
lotr_graph_window_100/num_edges 27 ns/iter (± 0) 8 ns/iter (± 0) 3.38
lotr_graph_window_100/num_nodes 90 ns/iter (± 0) 5 ns/iter (± 0) 18
lotr_graph_window_100_materialise/materialize 3412155 ns/iter (± 11335) 1616091 ns/iter (± 5804) 2.11
lotr_graph_window_10/iterate nodes 32485 ns/iter (± 79) 13145 ns/iter (± 247) 2.47
lotr_graph_window_10/iterate edges 105127 ns/iter (± 445) 49732 ns/iter (± 628) 2.11
lotr_graph_subgraph_10pc/has_edge_existing 476 ns/iter (± 5) 94 ns/iter (± 2) 5.06
lotr_graph_subgraph_10pc/has_node_existing 230 ns/iter (± 1) 35 ns/iter (± 0) 6.57
lotr_graph_subgraph_10pc/iterate nodes 3655 ns/iter (± 13) 1145 ns/iter (± 3) 3.19
lotr_graph_subgraph_10pc_windowed/iterate nodes 3923 ns/iter (± 13) 1763 ns/iter (± 30) 2.23
lotr_graph_window_50_layered/num_nodes 97960 ns/iter (± 4467) 23723 ns/iter (± 2295) 4.13
lotr_graph_window_50_layered/has_node_existing 1294 ns/iter (± 207) 140 ns/iter (± 15) 9.24
lotr_graph_window_50_layered/max_id 106022 ns/iter (± 2727) 30054 ns/iter (± 341) 3.53
lotr_graph_window_50_layered/max_degree 1986476 ns/iter (± 13998) 301176 ns/iter (± 12201) 6.60
lotr_graph_window_50_layered/iterate nodes 246739 ns/iter (± 636) 22036 ns/iter (± 207) 11.20
lotr_graph_window_50_layered/iterate edges 214214 ns/iter (± 382) 94360 ns/iter (± 2143) 2.27
lotr_graph_window_50_layered/max_neighbour_degree 3605677 ns/iter (± 15121) 405979 ns/iter (± 4756) 8.88
lotr_graph_window_50_layered_materialise/materialize 7776929 ns/iter (± 8802) 3369999 ns/iter (± 23217) 2.31
lotr_graph_persistent_window_50_layered/num_edges_temporal 577313 ns/iter (± 3209) 213894 ns/iter (± 3659) 2.70
lotr_graph_persistent_window_50_layered/num_nodes 134175 ns/iter (± 19432) 33955 ns/iter (± 907) 3.95
lotr_graph_persistent_window_50_layered/has_node_existing 1666 ns/iter (± 500) 179 ns/iter (± 93) 9.31
lotr_graph_persistent_window_50_layered/max_id 146526 ns/iter (± 3357) 42737 ns/iter (± 2220) 3.43
lotr_graph_persistent_window_50_layered/max_degree 2883210 ns/iter (± 17532) 466749 ns/iter (± 14768) 6.18
lotr_graph_persistent_window_50_layered/iterate nodes 325856 ns/iter (± 951) 41506 ns/iter (± 354) 7.85
lotr_graph_persistent_window_50_layered/max_neighbour_degree 4973150 ns/iter (± 76278) 525340 ns/iter (± 4113) 9.47
lotr_graph_persistent_window_50_layered_materialise/materialize 13943193 ns/iter (± 25165) 5151712 ns/iter (± 94249) 2.71
lotr_graph/proto_decode 19811967 ns/iter (± 105407) 1911611 ns/iter (± 25778) 10.36
lotr_graph/proto_encode 9187490 ns/iter (± 150270) 1153254 ns/iter (± 20173) 7.97

This comment was automatically generated by workflow using github-action-benchmark.

@ljeub-pometry ljeub-pometry marked this pull request as ready for review December 19, 2025 13:18
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.

2 participants