Repository navigation
AST row select cannot reference edge alias properties after traversal #982
Copy link
Copy link
Closed
Description
Activity
- added 3 commits that reference this issue
on Mar 31, 2026 - added a commit that references this issue
on Apr 1, 2026 Benchmark update from
pyg-benchon 2026-04-01 after correcting the staged SNB graph identity bug on our side.Current measured
IS3/interactive-short-3state on the corrected harness:- backend:
GFQL - status:
partial - issue ref:
#982 - expected rows:
920 - actual rows:
920 - no row diff in the current artifact
- latest artifact:
results/runs/dgx-spark-snb-interactive-core-sf1-v05315-r2/
Important correction:
- this is not a new residual wrong-answer bug
- after the harness fix, the remaining benchmark-side
IS3limitation is that we still need the adapter two-pass workaround to projectr.creationDatealongside the friend node columns - the current probe note is:
Semantic scope: adapter_two_pass_workaround.Adapter workaround: fetch friend-node rows and matched KNOWS-edge rows separately, then join locally.
So from the benchmark side,
#982is now the top active GFQL issue to clear on the short-query floor.- backend:
Benchmark follow-up from
pyg-benchon 2026-04-01.The benchmark-side signal now agrees with the local repro note and PR
#1015:#982looks fixed on currentorigin/masterand no longer looks like the next runtime feature ask.What changed on our side:
- earlier we elevated
#982because the correctedv0.53.15floor still showedIS3onGFQLas a workaround-backed partial - after re-checking the current evidence plus your verification branch, the practical next step here is to land the regression coverage and close the issue, not do more runtime work on it
The current active benchmark-side GFQL ask is back to
#880.- earlier we elevated
- added a commit that references this issue
on Apr 1, 2026 - added a commit that references this issue
on Apr 1, 2026
Summary
On the AST/GFQL path, row projection can reference node aliases like
friend.firstName, but it currently fails when projecting an edge alias property such asr.creationDateafter a traversal.Status
Partially fixed in v0.53.9 (PR #986).
The Cypher string path now works:
The native AST path (
rows()+select()) is still broken:The Cypher fix works via a bindings table that joins edges with node properties. The AST
rows()path doesn't go through the same bindings table construction — it would needrows(alias_endpoints=...)to be wired into the native chain execution, not just the Cypher lowering.Minimal repro (AST path — still failing)
Observed behavior
Expected behavior
[{'personId': 'b', 'firstName': 'Bob', 'friendshipCreationDate': 123}]Likely relation
Depends on wiring
rows(alias_endpoints=...)into the native AST chain execution path, not just the Cypher lowering path.