test(gfql): sibling-specialization sweep pins (#2051) - #2052
Merged
Conversation
… single-hop shape (#2051) Sibling-specialization sweep after #2046: polars' hop() collapses duplicate node rows (#1993) but the unnamed, untyped single-hop chain shape still keeps the duplicate. The collapsing shapes (named, typed, hops=2, undirected, hop()) are pinned green against pandas; the two leaking shapes are strict expected failures that flip with the fix. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01QztW7jYsDd66e8rb8pJNQA
lmeyerov
force-pushed
the
test/gfql-sibling-sweep-pins
branch
from
September 5, 2026 21:04
ea1f455 to
40a7d64
Compare
Contributor
Author
…every lane and engine The #2039 class probed on the routes the native collision matrix does not reach: the Cypher seeded typed-hop and node-lookup lanes, whole-entity return, the two-hop count lane, the lowered full path, and the native rows route — all match the pandas full path on pandas, cuDF and polars, scan and indexed. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01QztW7jYsDd66e8rb8pJNQA
Base automatically changed from
fix/gfql-2039-polars-alias-collision
to
master
September 5, 2026 21:43
Contributor
Author
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Test-only, stacked on #2046 (retargets to master when it lands).
Owner ask after #2046: a fix for one specialization may be needed on its siblings; audit the recent fixes. The sweep (25 merged fix PRs inventoried; the engine- or lane-specific ones re-probed on pandas, cuDF and polars × scan/indexed × native/Cypher/hop/multi-hop against the pandas full path; report in the campaign plan under
spec-sweep/) found one more miss beyond the #2039 family already handled in #2046:hop()de-dups its node table): the unnamed, untyped single-hop chain shape on polars keeps a duplicated node row; named, typed,hops=2, undirected andhop()collapse it as pandas does.This PR pins that contract in the polars mirror directory: the collapsing shapes green against pandas, the two leaking shapes as strict expected failures that flip with the fix. Registered in the polars test lane.
Classes re-probed clean on every engine × policy: #1990 EXISTS with indexes, #2000/#1991 bag multiplicity, #2002/#1982 boolean aggregates, #1998
IN/equality over categorical (string predicates over categorical raise the same E302 on every engine — consistent, flagged as a usability question in the report).🤖 Generated with Claude Code
https://claude.ai/code/session_01QztW7jYsDd66e8rb8pJNQA