Skip to content

Add fast path for last on properties#2584

Open
ljeub-pometry wants to merge 15 commits intodb_v4from
optimise_property_latest
Open

Add fast path for last on properties#2584
ljeub-pometry wants to merge 15 commits intodb_v4from
optimise_property_latest

Conversation

@ljeub-pometry
Copy link
Copy Markdown
Collaborator

What changes were proposed in this pull request?

Why are the changes needed?

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
Copy Markdown
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: a172391 Previous: 9823ef7 Ratio
lotr_graph/num_edges 5 ns/iter (± 0) 0 ns/iter (± 0) +∞
lotr_graph/num_nodes 127 ns/iter (± 31) 1 ns/iter (± 0) 127
lotr_graph/graph_latest 3 ns/iter (± 0) 0 ns/iter (± 0) +∞
lotr_graph_window_100/num_edges 29 ns/iter (± 0) 8 ns/iter (± 0) 3.63
lotr_graph_window_100/num_nodes 142 ns/iter (± 32) 5 ns/iter (± 0) 28.40
lotr_graph_window_100/has_node_existing 45 ns/iter (± 0) 22 ns/iter (± 0) 2.05
lotr_graph_window_10/has_node_existing 149 ns/iter (± 8) 62 ns/iter (± 11) 2.40
lotr_graph_window_10/iterate nodes 37362 ns/iter (± 85) 11339 ns/iter (± 40) 3.29
lotr_graph_window_10/iterate edges 97414 ns/iter (± 485) 48684 ns/iter (± 211) 2.00
lotr_graph_subgraph_10pc/has_edge_existing 264 ns/iter (± 11) 93 ns/iter (± 1) 2.84
lotr_graph_subgraph_10pc/num_nodes 124 ns/iter (± 26) 4 ns/iter (± 0) 31
lotr_graph_subgraph_10pc/has_node_existing 125 ns/iter (± 1) 34 ns/iter (± 0) 3.68
lotr_graph_subgraph_10pc/iterate nodes 2729 ns/iter (± 92) 839 ns/iter (± 5) 3.25
lotr_graph_subgraph_10pc_windowed/has_node_existing 142 ns/iter (± 8) 62 ns/iter (± 14) 2.29
lotr_graph_subgraph_10pc_windowed/iterate nodes 5115 ns/iter (± 97) 1365 ns/iter (± 3) 3.75
lotr_graph_window_50_layered/num_edges_temporal 154801 ns/iter (± 1956) 70121 ns/iter (± 7586) 2.21
lotr_graph_window_50_layered/num_nodes 63781 ns/iter (± 905) 21435 ns/iter (± 536) 2.98
lotr_graph_window_50_layered/has_node_existing 949 ns/iter (± 90) 129 ns/iter (± 12) 7.36
lotr_graph_window_50_layered/max_id 69898 ns/iter (± 1474) 25556 ns/iter (± 252) 2.74
lotr_graph_window_50_layered/iterate nodes 142339 ns/iter (± 435) 19308 ns/iter (± 47) 7.37
lotr_graph_window_50_layered/iterate edges 201745 ns/iter (± 496) 83616 ns/iter (± 1318) 2.41
lotr_graph_window_50_layered/graph_latest 104759 ns/iter (± 1196) 36649 ns/iter (± 916) 2.86
lotr_graph_persistent_window_50_layered/num_edges_temporal 726577 ns/iter (± 19639) 192686 ns/iter (± 1569) 3.77
lotr_graph_persistent_window_50_layered/num_nodes 82709 ns/iter (± 1535) 31517 ns/iter (± 779) 2.62
lotr_graph_persistent_window_50_layered/has_node_existing 1126 ns/iter (± 193) 174 ns/iter (± 83) 6.47
lotr_graph_persistent_window_50_layered/max_id 91863 ns/iter (± 1823) 38024 ns/iter (± 490) 2.42
lotr_graph_persistent_window_50_layered/iterate nodes 181598 ns/iter (± 692) 35886 ns/iter (± 191) 5.06
lotr_graph_persistent_window_50_layered/iterate edges 194293 ns/iter (± 713) 84161 ns/iter (± 596) 2.31
lotr_graph_persistent_window_50_layered/iterate_exploded_edges 3417801 ns/iter (± 8472) 1659940 ns/iter (± 19402) 2.06
lotr_graph_persistent_window_50_layered/graph_latest 161533 ns/iter (± 2029) 57549 ns/iter (± 4809) 2.81
lotr_graph_persistent_window_50_layered_materialise/materialize 12878994 ns/iter (± 66194) 5298035 ns/iter (± 147912) 2.43
lotr_graph/proto_encode 5770860 ns/iter (± 44238) 1157897 ns/iter (± 73709) 4.98

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

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