Skip to content

tests(gfql): scaling pin for undirected 2-hop (ratio vs two plain joins) - #2028

Merged
lmeyerov merged 1 commit into
masterfrom
test/gfql-hop-scaling-pin
Sep 5, 2026
Merged

lmeyerov merged 1 commit into
masterfrom
test/gfql-hop-scaling-pin

Conversation

@lmeyerov

@lmeyerov lmeyerov commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Stacked on #2024. Owner ask: regression/acceptance testing so a perf regression like #2023 cannot reach the docs unnoticed.

A ratio pin (robust to machine speed, unlike a wall-clock threshold): on a heavy-tailed 1.5M-edge graph, an undirected 2-hop chain from the 50 hub seeds must cost less than 30x two plain frame joins that traverse the same edges.

tree pandas polars
0.59.0 (3fb216dd, per-edge loop) 52.4x 47.2x — fails
#2024 head (engine-native rule) 20.2x 6.4x — passes

Runs in the core lanes (pandas) and the polars lane (listed in bin/test-polars.sh); ~25 s. Companion gates in pyg-bench: #233/#239 (sentinel lane + baseline), #241 (publication acceptance: unexplained regressions and index-parity gaps fail the export).

🤖 Generated with Claude Code

https://claude.ai/code/session_01WwMmVFo44ADiRRj5cxh7i1

…le of two plain joins

Ratio pin, not a wall-clock threshold: on a heavy-tailed 1.5M-edge graph
from 50 hub seeds, the 2-hop chain must cost < 30x two plain frame joins
that traverse the same edges. The 0.59.0 tree (per-edge Python loop, #2023)
measures 47-52x on pandas and polars; the engine-native rule 20x / 6x.

Co-Authored-By: Claude Fable 5.1 <[email protected]>
Claude-Session: https://claude.ai/code/session_01WwMmVFo44ADiRRj5cxh7i1
@lmeyerov
lmeyerov force-pushed the perf/gfql-undirected-seed-rediscovery branch from 4de8684 to fd1a0fb Compare September 5, 2026 07:34
@lmeyerov
lmeyerov force-pushed the test/gfql-hop-scaling-pin branch from b3d6677 to 962f690 Compare September 5, 2026 07:34
Base automatically changed from perf/gfql-undirected-seed-rediscovery to master September 5, 2026 07:55
@lmeyerov
lmeyerov merged commit bff7c32 into master Sep 5, 2026
76 checks passed
@lmeyerov
lmeyerov deleted the test/gfql-hop-scaling-pin branch September 5, 2026 07:56
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