Skip to content

Commit e8f4887

Browse files
authored
Analytics: create DB index on PageView.date (#10426)
We use this field to run cleaup tasks (`readthedocs.analytics.tasks.delete_old_page_counts`). Having an index on this field will help a lot to improve these queries. Related readthedocs/readthedocs-ops#1291
1 parent 9565578 commit e8f4887

File tree

2 files changed

+20
-1
lines changed

2 files changed

+20
-1
lines changed
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
# Generated by Django 3.2.19 on 2023-06-13 11:12
2+
3+
import datetime
4+
5+
from django.db import migrations, models
6+
7+
8+
class Migration(migrations.Migration):
9+
dependencies = [
10+
("analytics", "0006_alter_pageview_id"),
11+
]
12+
13+
operations = [
14+
migrations.AlterField(
15+
model_name="pageview",
16+
name="date",
17+
field=models.DateField(db_index=True, default=datetime.date.today),
18+
),
19+
]

readthedocs/analytics/models.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,7 @@ class PageView(models.Model):
8282
blank=True,
8383
)
8484
view_count = models.PositiveIntegerField(default=0)
85-
date = models.DateField(default=datetime.date.today)
85+
date = models.DateField(default=datetime.date.today, db_index=True)
8686
status = models.PositiveIntegerField(
8787
default=200,
8888
help_text=_("HTTP status code"),

0 commit comments

Comments
 (0)