Merge pull request rails#51815 from byroot/cached-find-by-connection-lease

byroot · web-flow · commit a8d26789a45e · 2024-05-14T15:20:07.000+09:00
Eliminate `lease_connection` call in `cached_find_by`
diff --git a/activerecord/lib/active_record/core.rb b/activerecord/lib/active_record/core.rb
@@ -432,7 +432,7 @@ def cached_find_by(keys, values)
             }
 
             begin
-              statement.execute(values.flatten, lease_connection, allow_retry: true).first
+              statement.execute(values.flatten, connection, allow_retry: true).first
             rescue TypeError
               raise ActiveRecord::StatementInvalid
             end

Original file line number	Diff line number	Diff line change
`@@ -432,7 +432,7 @@ def cached_find_by(keys, values)`
`432`	`432`	`}`
`433`	`433`
`434`	`434`	`begin`
`435`		`- statement.execute(values.flatten, lease_connection, allow_retry: true).first`
	`435`	`+ statement.execute(values.flatten, connection, allow_retry: true).first`
`436`	`436`	`rescue TypeError`
`437`	`437`	`raise ActiveRecord::StatementInvalid`
`438`	`438`	`end`