Permalink
Browse files

Define to_ary on ActiveRecord::Base to return nil. Improve performanc…

…e of Array#flatten under 1.9, see comment for details.
  • Loading branch information...
1 parent dc921ff commit b09ef6d139d02a073f826f068162724b10d2bf82 @jonleighton jonleighton committed Jun 29, 2011
Showing with 12 additions and 0 deletions.
  1. +12 −0 activerecord/lib/active_record/base.rb
@@ -1910,6 +1910,18 @@ def clone_attribute_value(reader_method, attribute_name)
private
+ # Under Ruby 1.9, Array#flatten will call #to_ary (recursively) on each of the elements
+ # of the array, and then rescues from the possible NoMethodError. If those elements are
+ # ActiveRecord::Base's, then this triggers the various method_missing's that we have,
+ # which significantly impacts upon performance.
+ #
+ # So we can avoid the method_missing hit by explicitly defining #to_ary as nil here.
+ #
+ # See also http://tenderlovemaking.com/2011/06/28/til-its-ok-to-return-nil-from-to_ary/
+ def to_ary # :nodoc:
+ nil
+ end
+
def set_serialized_attributes
sattrs = self.class.serialized_attributes

0 comments on commit b09ef6d

Please sign in to comment.