Skip to content

Set DEFAULT_NEW_STREAM_GRACE_PERIOD_MS to 0#1016

Merged
mittalprince merged 2 commits into
masterfrom
pmittal/default-grace-period-zero
May 20, 2026
Merged

Set DEFAULT_NEW_STREAM_GRACE_PERIOD_MS to 0#1016
mittalprince merged 2 commits into
masterfrom
pmittal/default-grace-period-zero

Conversation

@mittalprince
Copy link
Copy Markdown
Collaborator

Summary

  • Set DEFAULT_NEW_STREAM_GRACE_PERIOD_MS to 0 in EventProducer
  • Disables the SLA grace period by default so SLA metrics are emitted
    immediately for all new streams without a 2-hour warmup window

Testing Done

  • Build passes
  • Existing unit tests pass

Disables the SLA grace period by default so SLA metrics are emitted
immediately for all new streams without a warmup window.
@mittalprince mittalprince force-pushed the pmittal/default-grace-period-zero branch from 8852862 to 267def6 Compare May 20, 2026 15:40
…_MS=0

Tests that assert SLA suppression during grace period were relying on the
2h default. Now that the default is 0, pass an explicit 2h grace period
via props so they continue to test the intended behaviour.
@mittalprince mittalprince merged commit 394d7f6 into master May 20, 2026
1 check passed
@mittalprince mittalprince mentioned this pull request May 20, 2026
1 task
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.

4 participants