maverick/docbrown
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Update PageViewRollup (#20785)

Browse source 
* Make sure time is beginning of day

* Use correct initial time
This commit is contained in:
Mac Siri 2024-03-21 18:37:33 -04:00 • committed by GitHub
parent b0cc94e655
commit 7988af1628
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 4 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
app/services/page_view_rollup.rb
View file

@ -41,7 +41,7 @@ class PageViewRollup
end
def self.rollup(date, relation: PageView)
new(relation: relation).rollup(date.to_datetime)
new(relation: relation).rollup(date)
end
def initialize(relation:)
@ -52,10 +52,11 @@ class PageViewRollup
def rollup(date)
created = []
fixed_date = date.to_datetime.beginning_of_day
(0..23).each do |hour|
start_hour = date.change(hour: hour)
end_hour = date.change(hour: hour + 1)
start_hour = fixed_date.change(hour: hour)
end_hour = fixed_date.change(hour: hour + 1)
rows = relation.where(user_id: nil, created_at: start_hour...end_hour)
aggregate_into_groups(rows).each do |compacted_views|
created << compact_records(start_hour, compacted_views)