This project is archived and is in readonly mode.
Don't load ActiveRecord::SessionStore during intialization if it's not used by the app
Reported by Luke Melia | May 29th, 2009 @ 03:01 PM
Our app uses CookieStore but I noticed that ActiveRecord::SessionStore was still being loaded (with the DB reflection overhead associated with loading an AR class) during initialization. This patch just changes a conditional from using a constant to string to avoid autoloading the class.
Comments and changes to this ticket
-
josh May 29th, 2009 @ 03:32 PM
- Milestone cleared.
- State changed from “new” to “open”
- Assigned user set to “josh”
-
Repository May 30th, 2009 @ 03:37 PM
- State changed from “open” to “resolved”
(from [54984f0f245cd67999c23112dfa841a4d0bd66f2]) Avoid loading the ActiveRecord::SessionStore class on initialization if it is not in use [#2737 state:resolved]
Signed-off-by: Joshua Peek josh@joshpeek.com
http://github.com/rails/rails/commit/54984f0f245cd67999c23112dfa841... -
Repository July 27th, 2009 @ 11:16 PM
(from [60122e81a3504ed171a761573ed16cb034c79c13]) Avoid loading the ActiveRecord::SessionStore class on initialization if it is not in use [#2737 state:resolved]
Signed-off-by: Joshua Peek josh@joshpeek.com
http://github.com/rails/rails/commit/60122e81a3504ed171a761573ed16c...
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>
People watching this ticket
josh
Attachments
Tags
Referenced by
-
2737
Don't load ActiveRecord::SessionStore during intialization if it's not used by the app
(from [54984f0f245cd67999c23112dfa841a4d0bd66f2])
Avoid l...
-
2737
Don't load ActiveRecord::SessionStore during intialization if it's not used by the app
(from [60122e81a3504ed171a761573ed16cb034c79c13])
Avoid l...
-
2516
Avoid loading AR:SessionStore if not used
Indeed it was. It refers a different ticket though —
#2737