orneryd / NornicDB / 36356635416

87%

Build Type

push

github

Committed by orneryd

Commit Message

perf(cypher): a function context's graph catalog costs no allocation per row

The function context (cypherfn.Context) is built for every function call
on every row. #749 gave it the composite database's graphs as the method
value e.compositeGraphs, and a method value builds a closure each time,
so every function call on a row allocated once more
(TraversalOptionalMatch_FanOutProjection 6161 -> 6361,
GraphitiRelationshipVectorFastPath/indexed_relationship_vectors
2658 -> 2708). Context.Graphs is now a GraphCatalog interface, and the
executor (a pointer, stored without allocating) is the one catalog all
three contexts use.

TestFunctionContextGraphsAllocatesNothing pins that building a context
with the executor as its catalog allocates nothing.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Coverage Stats

1 of 1 new or added line in 1 file covered. (100.0%)

18 existing lines in 7 files now uncovered.

174736 of 200814 relevant lines covered (87.01%)

1.02 hits per line