diff options
author | Sam <sam.saffron@gmail.com> | 2018-06-06 09:50:58 +1000 |
---|---|---|
committer | Sam <sam.saffron@gmail.com> | 2018-06-06 09:50:58 +1000 |
commit | a46dcb7454b56c979cded85f2f4f875dcd2cfdf0 (patch) | |
tree | 7cb448f2d6764ecbfed081ab0263c6bbd726139b /activerecord/lib/active_record/attribute_methods.rb | |
parent | 32aa7cdd8f031dc756fdde3501efca7ff72bf576 (diff) | |
download | rails-a46dcb7454b56c979cded85f2f4f875dcd2cfdf0.tar.gz rails-a46dcb7454b56c979cded85f2f4f875dcd2cfdf0.tar.bz2 rails-a46dcb7454b56c979cded85f2f4f875dcd2cfdf0.zip |
PERF: avoid allocating column names where possible
When requesting columns names from database adapters AR:Result
would dup/freeze column names, this prefers using fstrings which
cuts down on repeat allocations
Attributes that are retained keep these fstrings around for the long
term
Note, this has the highest impact on "short" result sets, eg: Topic.first where you can void allocating the number of columns * String.
Diffstat (limited to 'activerecord/lib/active_record/attribute_methods.rb')
-rw-r--r-- | activerecord/lib/active_record/attribute_methods.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/activerecord/lib/active_record/attribute_methods.rb b/activerecord/lib/active_record/attribute_methods.rb index 83b5a5e698..e4b8b1a330 100644 --- a/activerecord/lib/active_record/attribute_methods.rb +++ b/activerecord/lib/active_record/attribute_methods.rb @@ -26,7 +26,7 @@ module ActiveRecord def self.set_name_cache(name, value) const_name = "ATTR_#{name}" unless const_defined? const_name - const_set const_name, value.dup.freeze + const_set const_name, -value end end } |