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

Patch in Rails fix for Rails 7.1 memory consumption increase, prior to it's release in presumably Rails 7.1.3 #2466

Merged
merged 1 commit into from Dec 12, 2023

Conversation

jrochkind
Copy link
Contributor

See rails/rails#50298

And ref #2449

@jrochkind
Copy link
Contributor Author

While our RAM usage has overall been much lower on average since we deployed the changes to reduce allocations -- we did just get some more R14's this morning.

I'm going to merge and deploy this and see how it effects our average and max/spiked RAM usage!

@jrochkind jrochkind merged commit e0c548f into master Dec 12, 2023
1 check passed
@jrochkind jrochkind deleted the patch_rails7.1_ram_leak branch December 12, 2023 18:37
@jrochkind
Copy link
Contributor Author

This definitely reduced our RAM consumption back to what was expected, averaging same as with Rails 7.0 deploy if not better.

jrochkind added a commit that referenced this pull request Feb 1, 2024
With

    bundle update

Includes update to latest Rails 7.1.3. Then we remove patch from #2466 that is no longer necessary in Rails 7.1.3.
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.

None yet

2 participants