Permalink
Browse files

Reduce allocations when extracting AR models

  • Loading branch information...
1 parent f20f90f commit 14b23ee5fbe184187b33909411f3f28b953207d1 @SamSaffron SamSaffron committed Sep 10, 2013
Showing with 13 additions and 1 deletion.
  1. +13 −1 activerecord/lib/active_record/associations/join_dependency/join_part.rb
View
14 activerecord/lib/active_record/associations/join_dependency/join_part.rb
@@ -62,7 +62,19 @@ def column_names_with_alias
end
def extract_record(row)
- Hash[column_names_with_alias.map{|cn, an| [cn, row[an]]}]
+ # This code is performance critical as it is called per row.
+ # see: https://github.com/rails/rails/pull/12185
+ hash = {}
+
+ index = 0
+ length = column_names_with_alias.length
+ while index < length
+ column_name,alias_name = column_names_with_alias[index]
+ hash[column_name] = row[alias_name]
+ index += 1
+ end
+
+ hash
end
def record_id(row)

0 comments on commit 14b23ee

Please sign in to comment.