Leahnaya/mastodon
1
0
Fork 0
mirror of https://github.com/mastodon/mastodon.git synced 2026-08-16 00:16:36 -05:00
Code Issues Actions 3 Packages Projects Releases Wiki Activity

Fix performance of user-focused queries in admin dashboard (#39929)

Browse Source
This commit is contained in:
Claire
2026-07-23 12:32:59 +02:00
parent a8a6bfdf7b
commit 408b49afac
3 changed files with 13 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/lib/admin/metrics/measure/new_users_measure.rb
View File

@@ -27,7 +27,7 @@ class Admin::Metrics::Measure::NewUsersMeasure < Admin::Metrics::Measure::BaseMe
WITH new_users AS (
SELECT users.id
FROM users
WHERE date_trunc('day', users.created_at)::date = axis.period
WHERE users.account_id >= (date_part('epoch', date_trunc('day', axis.period)::date) * 1000)::bigint << 16 AND users.account_id < ((date_part('epoch', date_trunc('day', axis.period)::date + ('1 day')::interval)) * 1000)::bigint << 16
)
SELECT count(*) FROM new_users
) AS value

2
app/lib/admin/metrics/retention.rb
View File

@@ -76,7 +76,7 @@ class Admin::Metrics::Retention
WITH new_users AS (
SELECT users.id
FROM users
WHERE date_trunc(:frequency, users.created_at)::date = axis.cohort_period
WHERE users.account_id >= (date_part('epoch', date_trunc(:frequency, axis.cohort_period)::date) * 1000)::bigint << 16 AND users.account_id < ((date_part('epoch', date_trunc(:frequency, axis.cohort_period)::date + ('1' || :frequency)::interval)) * 1000)::bigint << 16
),
retained_users AS (
SELECT users.id

14
spec/lib/admin/metrics/measure/new_users_measure_spec.rb
View File

@@ -12,9 +12,17 @@ RSpec.describe Admin::Metrics::Measure::NewUsersMeasure do
describe '#data' do
context 'with user records' do
before do
3.times { Fabricate :user, created_at: 2.days.ago }
2.times { Fabricate :user, created_at: 1.day.ago }
Fabricate :user, created_at: 0.days.ago
travel_to 2.days.ago do
# We specify the `id` because `travel_to` doesn't affect the database
3.times { Fabricate :account, id: Mastodon::Snowflake.id_at(Time.now.utc) }
end
travel_to 1.day.ago do
# We specify the `id` because `travel_to` doesn't affect the database
2.times { Fabricate :account, id: Mastodon::Snowflake.id_at(Time.now.utc) }
end
Fabricate :user
end
it 'returns correct user counts' do