Skip to content

Execute external exact work through QueryPlan DAG edges - #582

Merged
zzylol merged 2 commits into
mainfrom
feat/execute-installed-runtime-dags
Sep 10, 2026
Merged

zzylol merged 2 commits into
mainfrom
feat/execute-installed-runtime-dags

Conversation

@zzylol

@zzylol zzylol commented Sep 10, 2026 •

Copy link
Copy Markdown
Contributor

Candidate-filtered exact work was prepared through a CandidateTopK-specific side path, so the installed QueryPlan graph did not govern its dependency ordering. This change adds an async physical DAG scheduler and executes language-neutral ExternalExact leaves from their declared input edges. For example, CandidateTopK now reads candidate membership first, passes it to the external exact leaf through CandidateMembership, and then composes both ordinary node outputs.

Legacy logical exact leaves remain readable for existing serialized plans. New plans use ExternalExact, reject an unavailable language adapter explicitly, retain request de-duplication, and preserve empty-candidate short-circuiting and request-size limits.

Validation:

  • cargo check -p data_plane
  • cargo test -p data_plane exact_subqueries --lib (9 passed)

@zzylol
zzylol force-pushed the refactor/persist-post-asap-dag branch from affc4ba to 6832db8 Compare September 10, 2026 16:48
@zzylol
zzylol force-pushed the feat/execute-installed-runtime-dags branch from 7d878b3 to 674add4 Compare September 10, 2026 16:55
@zzylol
zzylol changed the base branch from refactor/persist-post-asap-dag to main September 10, 2026 16:56
@zzylol
zzylol force-pushed the feat/execute-installed-runtime-dags branch from 674add4 to 2482893 Compare September 10, 2026 16:56
@zzylol
zzylol merged commit f805174 into main Sep 10, 2026
1 check passed
@zzylol
zzylol deleted the feat/execute-installed-runtime-dags branch September 21, 2026 18:40
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