Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set :puma_preload_app to false #104

Merged
merged 1 commit into from
May 17, 2015
Merged

Set :puma_preload_app to false #104

merged 1 commit into from
May 17, 2015

Conversation

rafaelgoulart
Copy link
Contributor

Since the default number of workers is zero, preload_app default config should be false

After reading the puma's README, I noticed they don't recommend to use preload_app if you're not using workers. I think it'd better to set the default to false.

General rule is to use preload_app when your workers die often and need fast starts. If you don’t have many workers, you probably should not use preload_app

Since the default number of workers is zero, preload_app default config should be false
seuros added a commit that referenced this pull request May 17, 2015
Set :puma_preload_app to false
@seuros seuros merged commit e4d5d93 into seuros:master May 17, 2015
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.

2 participants