Repository navigation
Use explicit bounded reservoir for 4K startup - #81
Merged
Merged
Conversation
TheCryptoDonkey
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Cold 4K startup can need more than 24 MiB when the original response remains owned alongside three useful futures. Pin RelaySwarm 25354e3cb843b606fc51cdb47a548e9d90d0d091 and explicitly reserve a 32 MiB encoded reservoir for eligible full-recording startup within the existing 64 MiB shared held budget. Other playback retains the 24 MiB default; cached/in-flight future bytes remain separately capped at 24 MiB. The SDK also preserves the contiguous fourth-slot successor instead of horizon-skipping it.
Update module cache versions, bundled provenance/notices and factual documentation. Object limits, per-object size, recovery headroom, deadlines, playback controls and committed Rust/WASM bytes are unchanged. No public TURN service or credential route is added.
Validation: SDK full swarm672/672, warm135/135, six focused budget cases, source regression RED→GREEN, site focused43/43 and local build pass. Independent review verified source archive/bundle hashes, unchanged WASM, runtime scope, notices and documentation. The single preceding source-only late-join check still missed its deadline; the final deployed page needs real-source playback qualification. No sustained playback, traffic savings or current physical-device acceptance is claimed.
Depends on forgesworn/relayswarm#48 completing CI and merging. Its prerequisite #47 is merged.