Bug #104776: dataviz, cellule filtre qui provoque une 504 - Combo - Redmine Entr’ouvert

Bug #104776

dataviz, cellule filtre qui provoque une 504

Added by Valentin Deniaud about 1 month ago. Updated about 1 month ago.

Status:

Solution déployée

Priority:

Normal

Assignee:

Valentin Deniaud

Target version:

Start date:

16 April 2025

Due date:

% Done:

Estimated time:

Patch proposed:

Planning:

Tags:

statistiques

Description

Certainement lié à #97672 puis #100990.

Dans #104768 on constate une perte d'accès à une page, en front et en back, à cause d'une erreur 504. Dupliquer cette page conserve le problème. Supprimer la cellule filtre résout le problème, et la recréer ne le fait pas réapparaître.

À partir d'une duplication de la page de ce ticket https://agents.demarches.angouleme.fr/manage/pages/73/, il faudrait comprendre ce qu'il se passe (vu les tickets précédents il doit y avoir une requête SQL qui se bloque)

Associated revisions

Revision fd7c7e34 (diff)
Added by Valentin Deniaud about 1 month ago

dataviz: use set to compute choices intersection (#104776)

History

Updated by Benjamin Dauvergne about 1 month ago

Aucun rapport avec les tickets pointés, c'est une boucle trop longue dans ChartFiltersForm:

440 B->                    dynamic_fields[field_name].dataviz_choices = [
441                          x for x in dynamic_fields[field_name].dataviz_choices if x in field.dataviz_choices
442                      ]

ici si dynamic_fields[field_name].dataviz_choices et field.dataviz_choices contiennent chacun 15000 choix on fait 225 000 000 comparaisons. Peut-être utiliser un set() au lieu d'un recherche linéaire dans une list() ou alors limiter les choix à 1000 en toute circonstance, ça n'a de toute façon pas de sens de proposer autant de choix pour des statistiques.