You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Process differential E2E advances event time with a future watermark #488
This makes the test fast, but it exercises acceptance of future event time rather than the normal collector/backend timing contract. It may hide bugs in real window close, allowed lateness, watermark progression, and query evaluation relative to wall clock. The whole-backend E2E already avoids this by waiting for the controller-selected window to actually end.
Reproduction
./scripts/e2e.sh differential
Location: data_plane/tests/promql_differential_process_e2e.rs, around lines 235-242.
Expected
The stable production-process differential should not require future-dated telemetry.
Preferred options:
wait for the one-second window to end, then emit a wall-clock watermark; or
expose and wait for an explicit ingest/window-commit signal without modifying event timestamps.
Done when
The test passes using realistic non-future timestamps and deterministically observes the committed window without a fixed blind sleep.
Fixed in PR #478 at commit 2c7a5ed: the production DDSketch differential now uses two real past event timestamps and no longer advances the watermark with a fabricated future observation. The stable differential test passes.
Activity
zzylol commentedon Sep 3, 2026
Fixed in PR #478 at commit 2c7a5ed: the production DDSketch differential now uses two real past event timestamps and no longer advances the watermark with a fabricated future observation. The stable differential test passes.