Skip to content

Switch to using Thin instead of Unicorn #667

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

Closed

Conversation

tgraham
Copy link
Member

@tgraham tgraham commented Sep 28, 2012

Fixes Issue #547

… end user to pick their own and keeps dev env from failing on windows.
@tgraham
Copy link
Member Author

tgraham commented Sep 28, 2012

I guess we'll have to agree to disagree because what might be considered sane for one person might not work for another so great leaps are being taken when assuming a default is sane.

I wouldn't want to classify all users as wanting unicorn and expecting them to deploy to heroku, by default.

@wilkie
Copy link
Contributor

wilkie commented Sep 28, 2012

Sane is what works in 90% of cases. Surely, webrick as a default production server is weird, right? And terrible for people who aren't rubyists and want to be able to deploy without having to edit anything.

@Valarissa
Copy link
Contributor

I would agree with thin as a good alternative to Unicorn that should be an effective default. Webrick is simply unusable as a production server.

@carols10cents
Copy link
Contributor

I pretty much merged this as part of merging #665. Thank you @tgraham!!! ⭐ ❤️

@steveklabnik
Copy link
Contributor

Bikeshed comment for puma here ;)

@steveklabnik
Copy link
Contributor

Given https://blog.heroku.com/archives/2013/2/16/routing_performance_update/ thin is actually significantly worse.

@wilkie
Copy link
Contributor

wilkie commented Feb 16, 2013

Worse than Unicorn? Well, yes, with Rails the way it is now that's true. That's why it originally had the comment "Strongly encouraged" :) But, it was part of our needs to run on Windows story, and this is the world we live in. Yanno? Our defaults were encouraged to be 'easy' and 'ubiquitous' with an assumption that nodes will not require severe concurrency and thus Unicorn is not as acceptable of a default. An extra commit was recommended here to switch to Unicorn when those assumptions are wrong, for instance, for a large-scale node such as the main rstat.us instance.

@steveklabnik
Copy link
Contributor

I submitted a PR specifically to discuss the issue further. ;)

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.

5 participants