diff options
author | Eugene Kenny <elkenny@gmail.com> | 2017-04-24 21:10:47 +0100 |
---|---|---|
committer | Eugene Kenny <elkenny@gmail.com> | 2017-07-06 14:21:07 +0100 |
commit | 6658e3746b236f84e27e711fced6a83b187ad2b1 (patch) | |
tree | 7b6fb771bdb70425016a6a8f59883d0ea8a773ba /activerecord/lib/active_record/relation/batches.rb | |
parent | c8ce3459648ce0f86646b564ce1c0bb16a4b48eb (diff) | |
download | rails-6658e3746b236f84e27e711fced6a83b187ad2b1.tar.gz rails-6658e3746b236f84e27e711fced6a83b187ad2b1.tar.bz2 rails-6658e3746b236f84e27e711fced6a83b187ad2b1.zip |
Skip query cache for in_batches and friends
The `find_each`, `find_in_batches` and `in_batches` APIs usually operate
on large numbers of records, where it's preferable not to load them all
into memory at once.
If the query cache is enabled, it will hold onto the query results until
the end of the execution context (request/job), which means the memory
used is still proportional to the total number of records. These queries
are typically not repeated, so the query cache isn't desirable here.
Diffstat (limited to 'activerecord/lib/active_record/relation/batches.rb')
-rw-r--r-- | activerecord/lib/active_record/relation/batches.rb | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/activerecord/lib/active_record/relation/batches.rb b/activerecord/lib/active_record/relation/batches.rb index ee1f25ec84..c7e4f8a88a 100644 --- a/activerecord/lib/active_record/relation/batches.rb +++ b/activerecord/lib/active_record/relation/batches.rb @@ -209,6 +209,7 @@ module ActiveRecord relation = relation.reorder(batch_order).limit(batch_limit) relation = apply_limits(relation, start, finish) + relation.skip_query_cache! # Retaining the results in the query cache would undermine the point of batching batch_relation = relation loop do |