aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/lib/active_record/attribute_methods.rb
diff options
context:
space:
mode:
authorSam <sam.saffron@gmail.com>2018-06-06 09:50:58 +1000
committerSam <sam.saffron@gmail.com>2018-06-06 09:50:58 +1000
commita46dcb7454b56c979cded85f2f4f875dcd2cfdf0 (patch)
tree7cb448f2d6764ecbfed081ab0263c6bbd726139b /activerecord/lib/active_record/attribute_methods.rb
parent32aa7cdd8f031dc756fdde3501efca7ff72bf576 (diff)
downloadrails-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.rb2
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
}