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

Standardize on and tune Puma #2

Open
dahlbyk opened this issue May 17, 2016 · 0 comments
Open

Standardize on and tune Puma #2

dahlbyk opened this issue May 17, 2016 · 0 comments

Comments

@dahlbyk
Copy link
Member

dahlbyk commented May 17, 2016

https://huboard.slack.com/archives/development/p1463435226000084

@rauhryan:
I do believe that pubsub is running puma in enterprise

@drusellers:
I was just digging through pubsub code (which is pretty tiny) and was bouncing between the 2 real files (We could probably collapse into just one) and wasn’t sure if thin was correct anymore

if it is puma, we may want to put a puma config here too. Again, the goal is to just be clear about what systems are using what

Also, reading more at https://github.com/faye/faye-websocket-ruby (CMD+F puma)

it seems that thin and puma have different configurations. Might be worth double checking that all is well

Its such a small code base seems like we could ‘clean it up’ pretty quickly and be done with it.

I doubt pubsub has any perf issues that we could really tune but any tuning option we can would be nice

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants