Skip to content

S4: reduce CompiledCypher compatibility surface (#1305)#1307

Merged
lmeyerov merged 1 commit intomasterfrom
issue-1305-s4-compiledcypher-surface
May 5, 2026
Merged

S4: reduce CompiledCypher compatibility surface (#1305)#1307
lmeyerov merged 1 commit intomasterfrom
issue-1305-s4-compiledcypher-surface

Conversation

@lmeyerov
Copy link
Copy Markdown
Contributor

@lmeyerov lmeyerov commented May 5, 2026

Summary

  • reduce CompiledCypher compatibility accessor surface by retiring unused derived properties
  • rewire active compile/runtime call sites to read canonical nested execution_extras payloads
  • update lowering tests to assert canonical surfaces and add changelog entry

Surface reductions

  • CompiledCypherQuery: removed properties connected_optional_match, connected_match_join, query_graph, scope_stack, logical_plan_route
  • CompiledGraphBinding: removed logical_plan_route
  • CompiledCypherGraphQuery: removed logical_plan_route

Validation

  • ./bin/ruff.sh graphistry/compute/gfql/cypher/lowering.py graphistry/compute/gfql/cypher/reentry/runtime.py graphistry/compute/gfql_unified.py graphistry/tests/compute/gfql/cypher/test_lowering.py
  • ./bin/pytest.sh graphistry/tests/compute/gfql/cypher/test_lowering.py -q
  • ./bin/pytest.sh graphistry/tests/compute/test_gfql.py -q

Refs #1305
Parent #1260

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