This project is archived and is in readonly mode.

Changeset [b2600bfc181664fcfe448d100ca054017b0576dd] by José Valim

October 7th, 2010 @ 08:31 PM

Remove locals dependency from template.

This means that templates does not need to store its source anymore, allowing us to reduce the ammount of memory taken by our Rails processes. Naively speaking, if your app/views contains 2MB of files, each of your processes (after being hit by a bunch of requests) will take 2MB less of memory after this commit.

This is extremely important for the upcoming features. Since Rails will also render CSS and JS files, their source won't be stored as well allowing us to decrease the ammount of memory taken.
http://github.com/rails/rails/commit/b2600bfc181664fcfe448d100ca054...

Committed by José Valim

  • M actionpack/lib/action_view/lookup_context.rb
  • M actionpack/lib/action_view/paths.rb
  • M actionpack/lib/action_view/render/layouts.rb
  • M actionpack/lib/action_view/render/partials.rb
  • M actionpack/lib/action_view/render/rendering.rb
  • M actionpack/lib/action_view/template.rb
  • M actionpack/lib/action_view/template/resolver.rb
  • M actionpack/lib/action_view/template/text.rb
  • M actionpack/test/template/template_test.rb

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

<h2 style="font-size: 14px">Tickets have moved to Github</h2>

The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>