From e9d2e2baad82600e5f2bb7f7394df4f22f358e7e Mon Sep 17 00:00:00 2001
From: Fabre Florian <ffabre@hybird.org>
Date: Thu, 16 Oct 2025 16:20:03 +0200
Subject: [PATCH] =?UTF-8?q?=E2=9C=A8(backend)=20indexation=20tasks=20with?=
 =?UTF-8?q?=20throttle=20mechanism?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Add a celery task that send an item changes to the Find API
A simple flag is set in cache for an amount of time that block any other task
creation and do the throttle.
The SEARCH_INDEXER_COUNTDOWN setting gives the number of seconds between tasks

Signed-off-by: Fabre Florian <ffabre@hybird.org>
---
 src/backend/core/signals.py      | 31 +++++++++++
 src/backend/core/tasks/search.py | 88 ++++++++++++++++++++++++++++++++
 2 files changed, 119 insertions(+)
 create mode 100644 src/backend/core/signals.py
 create mode 100644 src/backend/core/tasks/search.py

diff --git a/src/backend/core/signals.py b/src/backend/core/signals.py
new file mode 100644
index 00000000..03e8df5c
--- /dev/null
+++ b/src/backend/core/signals.py
@@ -0,0 +1,31 @@
+"""
+Declare and configure the signals for the impress core application
+"""
+
+from functools import partial
+
+from django.db import transaction
+from django.db.models import signals
+from django.dispatch import receiver
+
+from . import models
+from .tasks.search import trigger_file_indexer
+
+
+@receiver(signals.post_save, sender=models.Item)
+def document_post_save(sender, instance, **kwargs):  # pylint: disable=unused-argument
+    """
+    Asynchronous call to the document indexer at the end of the transaction.
+    Note : Within the transaction we can have an empty content and a serialization
+    error.
+    """
+    transaction.on_commit(partial(trigger_file_indexer, instance))
+
+
+@receiver(signals.post_save, sender=models.ItemAccess)
+def document_access_post_save(sender, instance, created, **kwargs):  # pylint: disable=unused-argument
+    """
+    Asynchronous call to the document indexer at the end of the transaction.
+    """
+    if not created:
+        transaction.on_commit(partial(trigger_file_indexer, instance.item))
diff --git a/src/backend/core/tasks/search.py b/src/backend/core/tasks/search.py
new file mode 100644
index 00000000..7305cb4f
--- /dev/null
+++ b/src/backend/core/tasks/search.py
@@ -0,0 +1,88 @@
+"""Trigger document indexation using celery task."""
+
+from logging import getLogger
+
+from django.conf import settings
+from django.core.cache import cache
+
+from django_redis.cache import RedisCache
+
+from core import models
+from core.services.search_indexers import (
+    get_batch_accesses_by_users_and_teams,
+    get_file_indexer,
+)
+
+from drive.celery_app import app
+
+logger = getLogger(__file__)
+
+
+def indexer_throttle_acquire(document_id, timeout=0, atomic=True):
+    """
+    Enable the task throttle flag for a delay.
+    Uses redis locks if available to ensure atomic changes
+    """
+    key = f"file-indexer-throttle-{document_id}"
+
+    if isinstance(cache, RedisCache) and atomic:
+        with cache.locks(key):
+            return indexer_throttle_acquire(document_id, timeout, atomic=False)
+
+    # Use add() here :
+    #   - set the flag and returns true if not exist
+    #   - do nothing and return false if exist
+    return cache.add(key, 1, timeout=timeout)
+
+
+@app.task
+def file_indexer_task(item_id):
+    """Celery Task : Sends indexation query for a document."""
+    indexer = get_file_indexer()
+
+    if indexer is None:
+        return
+
+    try:
+        item = models.Item.objects.get(
+            pk=item_id,
+            deleted_at__null=True,
+            upload_state=models.ItemUploadStateChoices.READY,
+        )
+    except models.Item.DoesNotExist:
+        # Skip the task if the document does not exist.
+        return
+
+    accesses = get_batch_accesses_by_users_and_teams((item.path,))
+
+    data = indexer.serialize_item(item=item, accesses=accesses)
+
+    logger.info("Start file %s indexation", item_id)
+    indexer.push(data)
+
+
+def trigger_file_indexer(item):
+    """
+    Trigger indexation task with debounce a delay set by the SEARCH_INDEXER_COUNTDOWN setting.
+
+    Args:
+        item (Item): The file item instance.
+    """
+    countdown = settings.SEARCH_INDEXER_COUNTDOWN
+
+    # DO NOT create a task if indexation if disabled
+    if not settings.SEARCH_INDEXER_CLASS:
+        return
+
+    # Each time this method is called during a countdown, we increment the
+    # counter and each task decrease it, so the index be run only once.
+    if indexer_throttle_acquire(item.pk, timeout=countdown):
+        logger.info(
+            "Add task for file %s indexation in %.2f seconds",
+            item.pk,
+            countdown,
+        )
+
+        file_indexer_task.apply_async(args=[item.pk], countdown=countdown)
+    else:
+        logger.info("Skip task for file %s indexation", item.pk)