-
-
Notifications
You must be signed in to change notification settings - Fork 409
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
Improve Laravel integration performance #103
Comments
Hello! Here I did the integration back in February, ready for production and is actively used there. Can anyone check? https://github.com/Hunternnm/laravel-roadrunner Example https://github.com/Hunternnm/laravel-roadrunner-example |
A nice simple package for integrating Laravel with RoadRunner. I wanted to write my integration package, but this package is enough for me. There are no problems with the sessions, nor with any other difficulties. For more than 1.5 months, the site has been working in production, there are no problems with this package https://github.com/Hunternnm/laravel-roadrunner. Everything works perfectly. |
@Hunternnm Can confirm that your package works great. It needs more care due to the use of resetters, but once the config is taken care of, I'm seeing the performance boost I was expecting to get. (I did not see any perf improvement with https://github.com/avto-dev/roadrunner-laravel) I still need to confirm it in production, but this feels like the right approach. |
Feel free to add integration badges and notes here: https://github.com/spiral/roadrunner-docs/blob/master/integration/laravel.md I assume we can close this issue now :) |
Guys, please take a loot at this solution https://github.com/swooletw/laravel-swoole/blob/master/config/swoole_http.php#L92-L120
Looks like to make roadrunner integration with Laravel to be most flexible and fast, it should implement same logics (service providers and instances restarters + cached instances sandbox).
Thank you!
Originally posted by @barbushin in #96 (comment)
The text was updated successfully, but these errors were encountered: