Project

General

Profile

Bug #8515

Problème au sein de l'API ajax New Scorm.

Added by Michèle S. over 2 years ago. Updated over 2 years ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
Tracking / Reporting
Target version:
-
Start date:
13/02/2017
Due date:
% Done:

0%

Estimated time:
Complexity:
Normal
SCRUM pts - complexity:
?

Description

Je pense que nous avons détecté un problème au sein de l'API ajax New Scorm.

Les performances se dégradent drastiquement lors de l'appel :
chamilo/main/newscorm/lp_ajax_save_item.php
nous avons des temps de réponse pouvant aller jusqu'à plus de 60s

En analysant il s'avère que c'est $myLP->get_complete_items_count() qui provoque le ralentissement.
(learnpath::get_complete_items_count, learnpathItem::get_status)

N'y a t-il pas un problème de conception sur cette boucle effectuant une requête en base sur tous les items du parcours ?
La table c_lp_item_view pouvant être très conséquente > 500 000 ligne ne faut il pas revoir ce fonctionnement ?

Pouvez vous nous confirmer qu'il y a bien un défaut sur ce point ?

Nous avons pour la méthode status_is désactivé les requêtes via le flag de la methode get_status
$currentStatus = $this->get_status(true); => $currentStatus = $this->get_status(false);
ce qui nous a permis de revenir à des temps de réponse corrects.

Quel impact cela peut-il avoir sur le calcul des statistiques ?
Quel(s) autre(s) réglage(s) ou action pouvons nous mener pour retrouver un niveau de temps de réponse acceptable ?

--
tests effectués sur la version 1.11.2

History

#1

Updated by Yannick Warnier over 2 years ago

  • Priority changed from Urgent to Normal

Bonjour Michèle,

Nous transitons doucement vers https://github.com/chamilo/chamilo-lms/issues pour le rapport d'erreurs/améliorations. Il vaut donc mieux enregistrer ceci à cet endroit.

Avant de jeter un oeil plus en détail sur ce que tu indiques (qui m'a l'air intéressant et exploitable), je trouve un peu perturbant que tu rapportes des tests sur 1.11.2 dans le répertoire main/newscorm/ alors que celui-ci a changé de nom dans 1.11.0 vers main/lp/
Pourrais-tu dès lors me confirmer la version, histoire de ne pas travailler inutilement?

#2

Updated by Michèle S. over 2 years ago

Merci pour ce retour,

concerant le nom du répertoire effectivement nous vous avons communiqué celui de la version 1.10.8
(version sur laquelle nous avons initiallement détecté le problème)

Nous vous confirmons que nous avons bien mené les mêmes tests sur la version 1.11.2.
Nous reproduisons l'anomalie pour cette version (la montée de version ne corrige pas le défaut).

Je prends note du changement de plate-forme pour le rapport d'erreurs/améliorations.

#3

Updated by Michèle S. over 2 years ago

Merci pour ce retour,

concerant le nom du répertoire effectivement nous vous avons communiqué celui de la version 1.10.8
(version sur laquelle nous avons initiallement détecté le problème)

Nous vous confirmons que nous avons bien mené les mêmes tests sur la version 1.11.2.
Nous reproduisons l'anomalie pour cette version (la montée de version ne corrige pas le défaut).

Je prends note du changement de plate-forme pour le rapport d'erreurs/améliorations.

Also available in: Atom PDF