perf: add index to is_blacklisted and created field for performance i…

…mprovements for xblocks api

taxonomy/api/v1/views.py

@@ -297,7 +297,6 @@ class XBlockSkillsViewSet(TaxonomyAPIViewSetMixin, RetrieveModelMixin, ListModel
     If skill validation is disabled for a course, then return an empty queryset.
     """
     serializer_class = XBlocksSkillsSerializer
-    permission_classes = (permissions.IsAuthenticated, )
     filter_backends = (DjangoFilterBackend,)
     filterset_class = XBlocksFilter
 

taxonomy/migrations/0037_alter_xblockskilldata_is_blacklisted_and_more.py

@@ -0,0 +1,22 @@
+# Generated by Django 4.2.14 on 2024-08-22 09:43
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('taxonomy', '0036_auto_20240325_1631'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='xblockskilldata',
+            name='is_blacklisted',
+            field=models.BooleanField(db_index=True, default=False, help_text='Blacklist this xblock skill, useful to handle false positives.'),
+        ),
+        migrations.AddIndex(
+            model_name='xblockskilldata',
+            index=models.Index(fields=['created'], name='taxonomy_xb_created_5929ec_idx'),
+        ),
+    ]

taxonomy/models.py

@@ -238,7 +238,7 @@ class XBlockSkillData(TimeStampedModel):
     )
     is_blacklisted = models.BooleanField(
         help_text=_('Blacklist this xblock skill, useful to handle false positives.'),
-        default=False,
+        default=False, db_index=True
     )
 
     class Meta:
@@ -251,6 +251,9 @@ class Meta:
         ordering = ('created', )
         app_label = 'taxonomy'
         unique_together = ('xblock', 'skill')
+        indexes = [
+            models.Index(fields=['created']),
+        ]
 
     def __str__(self):
         """