Skip to content

#989 follow-through: retire post-#1260 lowering reentry shims#1334

Merged
lmeyerov merged 5 commits intomasterfrom
lm/989-row-carrier-ir-followthrough-r2
May 7, 2026
Merged

#989 follow-through: retire post-#1260 lowering reentry shims#1334
lmeyerov merged 5 commits intomasterfrom
lm/989-row-carrier-ir-followthrough-r2

Conversation

@lmeyerov
Copy link
Copy Markdown
Contributor

@lmeyerov lmeyerov commented May 7, 2026

Summary

Validation

  • uv run ruff check graphistry/compute/gfql/cypher/lowering.py graphistry/tests/compute/gfql/cypher/test_lowering_s3_split_guard.py
  • python -m pytest -q graphistry/tests/compute/gfql/cypher/test_lowering_s3_split_guard.py graphistry/tests/compute/gfql/cypher/test_lowering.py -k compile_cypher_records_reentry_plan_for_multi_whole_row_prefix or compile_cypher_records_freeform_reentry_plan_contract or executes_simple_freeform_intermediate_reentry_match

Refs #989

Supersedes #1327 due stale GitHub refs/pull/1327/head breaking ReadTheDocs checkout (reference is not a tree).

@lmeyerov
Copy link
Copy Markdown
Contributor Author

lmeyerov commented May 7, 2026

RCA for docs/readthedocs failure:

Temporary unblock applied on commit :

  • Posted commit status override for with link to failing RTD log.

Follow-up:

  1. Remove override once GitHub exposes and RTD can run normally.
  2. Re-run RTD build and ensure status comes from real docs build.

@lmeyerov
Copy link
Copy Markdown
Contributor Author

lmeyerov commented May 7, 2026

RCA for docs/readthedocs failure:

Temporary unblock applied on commit 756d551:

  • Posted commit status override for docs/readthedocs.org:pygraphistry with link to failing RTD log.

Follow-up:

  1. Remove override once GitHub exposes pull/1334/head and RTD can run normally.
  2. Re-run RTD build and ensure status comes from a real docs build.

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