chiu
/
impressionist
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #37 from challengepost/optimize_impressionist_count 

Optimize impressionist_count query via arel count with distinct option
This commit is contained in:
Erik Michaels-Ober 2012-03-21 12:47:18 -07:00
parent 6e6016480b fcfacdfdc9
commit 95125f4627
2 changed files with 2 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
app/models/impressionist/impressionable.rb
View File

@ -26,10 +26,7 @@ module Impressionist
def impressionist_count(options={})
options.reverse_merge!(:filter=>:request_hash, :start_date=>nil, :end_date=>Time.now)
imps = options[:start_date].blank? ? impressions : impressions.where("created_at>=? and created_at<=?",options[:start_date],options[:end_date])
if options[:filter]!=:all
imps = imps.select(options[:filter]).group(options[:filter])
end
imps.all.size
options[:filter] == :all ? imps.count : imps.count(options[:filter], :distinct => true)
end
def update_counter_cache

2
test_app/spec/models/counter_caching_spec.rb
View File

@ -21,7 +21,7 @@ describe Impression do
describe "#update_counter_cache" do
it "should update the counter cache column to reflect the correct number of impressions" do
lambda {
Impression.create(:impressionable_type => @widget.class.name, :impressionable_id => @widget.id)
@widget.impressions.create(:request_hash => 'abcd1234')
@widget.reload
}.should change(@widget, :impressions_count).from(0).to(1)
end