Repository navigation
0.1.8: four uploads at once for each server - #4
Merged
Merged
Conversation
Every system sending to the same server shares one lane of the queue, and the SDK's default of 2 workers leaves each upload ~1.5 s at 4800 calls/hr before calls start to wait and reach Broadcastify Calls late. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
Raises the upload queue's workers from the SDK default of 2 to 4 per server (
UPLOAD_THREADS).Why. Every system that sends to the same server shares one lane of the queue, so with the default all of them share 2 workers. A user on a heavy setup reported about 4800 calls/hr. At that rate each upload has about 1.5 s on average before calls start to wait, which shows up as late delivery on Broadcastify Calls. Broadcastify's upload is two round trips (the details POST, then the audio PUT to S3), so it hits that limit first.
Call timestamps don't change: a backlog only delays delivery.
The change is in this plugin rather than the SDK's
QueueOptions::default, so other plugins keep their own defaults.cargo test🤖 Generated with Claude Code