insertDeletion looks up recognize_fs_deletions by node_id alone, which no index covers — bulk removals scan the whole table per file
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 82/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- mariadb, php
- Área
- backend, databases, performance
Línea de trabajo
Empieza en lib/Db/FsActionMapper.php con findByNodeId() e insertDeletion() y, a continuación, inspecciona el índice unique existente de lib/Migration/Version011000001Date20251215094821.php. Confirma que la consulta utiliza la cobertura disponible de storage_id y node_id, o evalúa la alternativa de migración de node_id. Se considera terminado cuando las eliminaciones masivas evitan un escaneo completo del índice por archivo; utiliza EXPLAIN y el escenario files:scan reportado para verificar el plan de consulta.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
FileListener::postDelete() calls FsActionMapper::insertDeletion() for every file under a removed node. insertDeletion() first runs findByNodeId(), i.e. SELECT … FROM oc_recognize_fs_deletions WHERE node_id = ?. The table's only secondary index is the unique (storage_id, node_id) (recognize_fs_del_uniq), which cannot serve a node_id-only predicate, so each call reads the whole index.
On Nextcloud 34.0.2 / Recognize 12.0.2, MariaDB 11.4, with ~512,000 queued rows, an occ files:scan that removed 131,004 files ran at 8–20 files/s and would have taken 1–3 h (EXPLAIN showed type: index, ~539,000 rows per lookup). After ALTER TABLE oc_recognize_fs_deletions ADD INDEX (node_id) the same scan took 5 m 39 s.
Suggested fix: look the row up by (storage_id, node_id), which insertDeletion() already has and the existing unique index covers — or add a node_id index in a migration. The code is the same at the installed tag v12.0.2 (lib/Db/FsActionMapper.php findByNodeId() / insertDeletion(), lib/Migration/Version011000001Date20251215094821.php) and still present on main at dc45cc1.
- Lenguaje dominante
- PHP
- Estrellas
- 699
- Forks
- 68
- Merge medio
- 18 h 34 min
- PR fusionados (30 d)
- 2
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de nextcloud/recognize
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
enhancement
Dificultad 3/5 1-2 días Aptitud para principiantes 65/100
-
Model download button in UIAbiertoenhancement
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
-
Recognize maintenance routine unnecessarily redownloads Tensorflow CPU and GPU models every timeQuizá libre de nuevo @marcelklehr la tomó hace 188 días y no hay ningún pull request abierto. Abiertobug
-
enhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 38/100
Todos los issues de nextcloud/recognize
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
api-platform/core#8649 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 84/100
open-telemetry/opentelemetry-php#2071 ·
Los mantenedores suelen responder en 1 día
-
bug No Code Attached Yet
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
joomla/joomla-cms#48587 ·
Los mantenedores suelen responder en 1 día
-
priority:low type:docs
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
crazy-goat/rabbit-stream#662 ·
Los mantenedores suelen responder en 1 día
-
[Bug] Next service page errorAbiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
FOSSBilling/FOSSBilling#4498 ·
Los mantenedores suelen responder en 1 día