Project

General

Profile

Development #72784

FTS: ne pas indexer une chaîne en boucle

Added by Pierre Ducroquet 11 months ago. Updated 11 months ago.

Status:
Fermé
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
26 December 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

Vu sur chatellerault (suite bug #72776) : l'indexation FTS va mettre dans certains cas une même chaîne de caractère en boucle (à cause des objets Evolution attachés), engendrant un volume massif de données et donc des requêtes lentes.
Il ne me semble pas nécessaire d'envoyer ces données dupliquées.


Files

Associated revisions

Revision 25452fe9 (diff)
Added by Pierre Ducroquet 11 months ago

sql: don't add the same string multiple time in FTS (#72784)

History

#2

Updated by Frédéric Péters 11 months ago

Je verrais plutôt set() utilisé plus haut, dès

        fts_strings = {'A': [], 'B': [], 'C': [], 'D': []}

(je peux me charger du patch).

Cela étant, cela pointe peut-être également un problème dans le workflow qui boucle inutilement, tu pourrais par exemple donner l'adresse d'une demande qui présente ça ?

#3

Updated by Pierre Ducroquet 11 months ago

Frédéric Péters a écrit :

Je verrais plutôt set() utilisé plus haut, dès

[...]

Effectivement (je modifie ma branche dans ce sens)

Cela étant, cela pointe peut-être également un problème dans le workflow qui boucle inutilement, tu pourrais par exemple donner l'adresse d'une demande qui présente ça ?

2022-12-11 00:53:23.684 CET [3244227] wcs@wcs_formulaires_mesdemarches_grand_chatellerault_fr LOG: durée : 202.421 ms, instruction : UPDATE formdata_85_inscrire_un_enfant_2021_copie SET fts = ... WHERE id = 106

#4

Updated by Pierre Ducroquet 11 months ago

#7

Updated by Frédéric Péters 11 months ago

  • Status changed from Solution proposée to Résolu (à déployer)

validé poussé,

commit 25452fe91f0fca5751593f709d0c1aafaedea1a0
Author: Pierre Ducroquet <pducroquet@entrouvert.com>
Date:   Mon Dec 26 12:11:49 2022 +0100

    sql: don't add the same string multiple time in FTS (#72784)
#8

Updated by Transition automatique 11 months ago

  • Status changed from Résolu (à déployer) to Solution déployée
#9

Updated by Transition automatique 9 months ago

Automatic expiration

Also available in: Atom PDF