Project

General

Profile

Development #90533

wcs_search_token: revoir le mécanisme de purge

Added by Pierre Ducroquet 11 days ago. Updated 10 days ago.

Status:
En cours
Priority:
Normal
Target version:
-
Start date:
13 May 2024
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

Description

J'ai sous-estimé le changement en ajoutant les fiches à la requête de purge et j'aurais du mesurer sur nos plus grosses bases.
Plan:
1) désactiver le cron pour éviter que ça ne repasse et provoque des problèmes
2) revoir la requête en question (probablement avec un impact sur le stockage des tokens, j'y réfléchis en même temps)


Files


Related issues

Related to w.c.s. - Development #90534: wcs_search_token: désactiver temporairement la purge des tokensSolution déployée13 May 2024

Actions

Associated revisions

Revision a55d0823 (diff)
Added by Pierre Ducroquet 10 days ago

wcs_search_tokens: don't index numbers (#90533)

Revision db364ce0 (diff)
Added by Pierre Ducroquet 10 days ago

wcs_search_tokens: don't index numbers (#90533)

Revision e84db712 (diff)
Added by Pierre Ducroquet 10 days ago

wcs_search_tokens: don't index numbers (#90533)

Revision 900d0006 (diff)
Added by Pierre Ducroquet 10 days ago

wcs_search_tokens: don't index numbers (#90533)

History

#1

Updated by Pierre Ducroquet 11 days ago

  • Related to Development #90534: wcs_search_token: désactiver temporairement la purge des tokens added
#2

Updated by Pierre Ducroquet 11 days ago

La requête de DELETE atteignait une complexité bien trop importante avec les N tables de fiches, j'ai pensé et testé sur des instances avec un nombre faible de fiches, ma faute.
Donc elle prenait pour certaines instances un temps énorme, pouvant gêner la production avec des locks sur certaines lignes.

#4

Updated by Frédéric Péters (de retour le 27 mai) 11 days ago

  • Assignee set to Pierre Ducroquet
#5

Updated by Pierre Ducroquet 10 days ago

J'ai analysé sur une instance de recette, je n'ai pas de locks bloquant avec la requête de purge elle-même, par contre elle aura pu contribuer à l'incident en consommant des ressources côté front et des connexions, notamment en entrant en conflit avec le cron update_storage qui réclame des locks exclusifs sur beaucoup de tables, alors que la requête de purge aura pris un lock en accès partagé.
Je vais revoir le mécanisme de purge, et en profiter pour améliorer l'indexation suite à des observations que j'ai pu faire lors de l'analyse ici (un pourcentage non négligeable de la table, facilement 30%, contient des chiffres et donc jamais utilisés dans la fonction de recherche)

#6

Updated by Robot Gitea 10 days ago

  • Status changed from Nouveau to En cours

Pierre Ducroquet (pducroquet) a ouvert une pull request sur Gitea concernant cette demande :

Also available in: Atom PDF