I sped up Rails total response time by 5% with a 5 line change: https://github.com/rails/rails/pull/33835 … 
Show this thread
-
It's not 17% but geez. 11% performance bump for my OVERALL application speed from a 5 line change in a dependency (Rails) is way more than I was hoping for when I opened the patch.
Show this thread
I think the discussion around this PR has been a great example of the complexities of OSS honestly, I suspect we'll end up merging it, but a great example of where a change has a lot more tradeoffs than might be immediately apparent
-
-
100% agree. I don’t think anyone raised any concerns with the initial modification in 5.2.1. Got totally missed.
-
Did you mean 5.2.0? I'm honestly not sure I know about the issue with 5.2.1
- 5 more replies
New conversation -
Loading seems to be taking a while.
Twitter may be over capacity or experiencing a momentary hiccup. Try again or visit Twitter Status for more information.