Skip to content

DDIR: test shared-input joins through the server - #859

Merged
frankmcsherry merged 2 commits into
master-nextfrom
codex/ddir-prepared-requests
Sep 6, 2026
Merged

DDIR: test shared-input joins through the server#859
frankmcsherry merged 2 commits into
master-nextfrom
codex/ddir-prepared-requests

Conversation

@frankmcsherry

@frankmcsherry frankmcsherry commented Sep 6, 2026

Copy link
Copy Markdown
Member

DDIR: test shared-input joins through the server

Regression coverage only: no server code changes or new query capability.

Factor the existing integration test's process startup into a helper accepting
a worker count. The new scenario installs an arranged edge export and two
consumers that each join request rows (request_id, node) against it. Feed
requests and graph changes through TCP, then check peek after each tick.

This checks ordinary DD semantics through the server path: both consumers see
the shared input, retractions update the answers, equal bindings under distinct
IDs remain distinct, missing keys return no rows, and a reused ID gets the
answer for its new binding.

Four separately named tests run the same scenario on Vec and Corgi with one
and four workers, allowing parallel execution and identifying the failing
configuration directly. The README keeps only a short usage paragraph about
request rows and existing feed/tick/peek behavior.

Validation

From the worktree, using the existing pinned upstream Corgi:

CARGO_PROFILE_DEV_DEBUG=0 CARGO_PROFILE_TEST_DEBUG=0 CARGO_INCREMENTAL=0 cargo test -p ddir-server
CARGO_PROFILE_DEV_DEBUG=0 CARGO_PROFILE_TEST_DEBUG=0 CARGO_INCREMENTAL=0 cargo test -p ddir-server --test multiworker -- --test-threads=6
git diff --check c33b0fd0

The server suite passes, including the four backend/worker configurations and
the existing integration tests running concurrently. Existing workspace CI
discovers these tests. No new dependencies, data downloads, external database,
or performance thresholds.

@frankmcsherry frankmcsherry changed the title DDIR: test prepared requests against shared changing inputs DDIR: test shared-input joins through the server Sep 6, 2026
@frankmcsherry
frankmcsherry merged commit 9a02d5d into master-next Sep 6, 2026
6 checks passed
@frankmcsherry
frankmcsherry deleted the codex/ddir-prepared-requests branch September 6, 2026 20:35
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