Skip to content

Don't factor max system memory into individual SMS maxes#1431

Merged
acco merged 1 commit intomainfrom
acco/05-12-don_t_factor_in_system_max_memory_in_individual_sms_sizes
May 12, 2025
Merged

Don't factor max system memory into individual SMS maxes#1431
acco merged 1 commit intomainfrom
acco/05-12-don_t_factor_in_system_max_memory_in_individual_sms_sizes

Conversation

@acco
Copy link
Contributor

@acco acco commented May 12, 2025

We let you set a max memory for Sequin (defaults to ~80% of avail memory) or for individual slot message stores.

Prior behavior was to do min( max_memory/count_sms , sms_max ). This change removes that - we'll let an SMS fill up to his max, so long as the system can support it.

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label May 12, 2025
Copy link
Contributor Author

acco commented May 12, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@acco acco changed the title make signoff Don't factor max system memory into individual SMS maxes May 12, 2025
We let you set a max memory for Sequin (defaults to ~80% of avail memory) or for individual slot message stores.

Prior behavior was to do min( max_memory/count_sms , sms_max ). This change removes that - we'll let an SMS fill up to his max, so long as the system can support it.
@acco acco force-pushed the acco/05-12-don_t_factor_in_system_max_memory_in_individual_sms_sizes branch from 8f0efea to d0ad929 Compare May 12, 2025 21:35
@acco acco merged commit a5c226c into main May 12, 2025
3 checks passed
@acco acco deleted the acco/05-12-don_t_factor_in_system_max_memory_in_individual_sms_sizes branch May 12, 2025 21:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant