diff options
author | Godfrey Chan <godfreykfc@gmail.com> | 2014-05-22 11:58:20 -0700 |
---|---|---|
committer | Godfrey Chan <godfreykfc@gmail.com> | 2014-05-22 14:35:15 -0700 |
commit | 28d52c59f2cb32180ca24770bf95597ea3ad8198 (patch) | |
tree | df363f449a9ce95dc018712d43929d80c9fca202 /activesupport/test/xml_mini/rexml_engine_test.rb | |
parent | 2d73f5ae2fe75427d56853f107abda21754f1b1a (diff) | |
download | rails-28d52c59f2cb32180ca24770bf95597ea3ad8198.tar.gz rails-28d52c59f2cb32180ca24770bf95597ea3ad8198.tar.bz2 rails-28d52c59f2cb32180ca24770bf95597ea3ad8198.zip |
Avoid slowing down AR object initialization
2d73f5a forces AR to enter the `define_attribute_methods` method whenever it
instantiate a record from the `init_with` entry point. This is a potential
performance hotspot, because `init_with` is called from all `find*` family
methods, and `define_attribute_methods` is slow because it tries to acquire
a lock on the mutex everytime it is entered.
By using [DCL](http://en.wikipedia.org/wiki/Double-checked_locking), we can
avoid grabbing the lock most of the time when the attribute methods are already
defined (the common case). This is made possible by the fact that reading an
instance variable is an atomic operation in Ruby.
Credit goes to Aaron Patterson for pointing me to DCL and filling me in on the
atomicity guarantees in Ruby.
[*Godfrey Chan*, *Aaron Patterson*]
Diffstat (limited to 'activesupport/test/xml_mini/rexml_engine_test.rb')
0 files changed, 0 insertions, 0 deletions