Skip to content

[GFQL] Lock reply-author row-shaping coverage#1425

Merged
lmeyerov merged 1 commit into
masterfrom
issue-1412-reply-author-row-shaping
May 11, 2026
Merged

[GFQL] Lock reply-author row-shaping coverage#1425
lmeyerov merged 1 commit into
masterfrom
issue-1412-reply-author-row-shaping

Conversation

@lmeyerov
Copy link
Copy Markdown
Contributor

Summary

  • Add native GFQL rows() regression coverage for the [POST-RERUN] #880 residual: reply-author row-shaping bucket #1412 SNB reply-author row-shaping residual bucket.
  • Cover both affected shapes: IC8 recent-replies and IS7 message-replies.
  • Assert parity between bare native rows() injection and explicit rows(binding_ops=...), locking the pygraphistry-side behavior needed to retire adapter-local reply-author joins in pyg-bench.

Refs #1412

Validation

  • python3 -m pytest -q graphistry/tests/test_compute_chain.py -k 'issue_1412'
  • python3 -m pytest -q graphistry/tests/test_compute_chain.py -k 'issue_1412 or native_chain_rows_bindings_with_select'
  • ./bin/ruff.sh graphistry/tests/test_compute_chain.py
  • git diff --check

Notes

  • Broad TestChainBindingsTable local run is blocked by environment gaps unrelated to this patch: no CUDA device for cuDF tests and missing lark for row-expression parser tests.
  • pyg-bench still needs the adapter workaround removal and fresh rerun before [POST-RERUN] #880 residual: reply-author row-shaping bucket #1412 can be fully closed.

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