In the logs, there's many "SELECT Context, GroupDirectory_Id FROM GroupDirectoriesByContext WHERE Context=?" SQL queries.
It may be related, or not ...
Est composé de
RUNTIME-3681Add a cache in GroupDirectoryContextHelper
le première est là pour retirer l'argument de tri de la méthode getProjectMembers qui avait été ajouté pour des perfs
la deuxième est l'ajout du cache sur la méthode getProjectMembers. L'objet du ticket
la troisième c'est du nettoyage de la classe ProjectMemberManager pour uniformiser les méthodes utilisées pour l'ajout d'un membre.
Revue fonctionnelle:
pas de revue fonctionnelle, le test sera un test de perf en prod.
Guillaume Gouin
added a comment - Revue technique:
La PR a 3 commit:
le première est là pour retirer l'argument de tri de la méthode getProjectMembers qui avait été ajouté pour des perfs
la deuxième est l'ajout du cache sur la méthode getProjectMembers. L'objet du ticket
la troisième c'est du nettoyage de la classe ProjectMemberManager pour uniformiser les méthodes utilisées pour l'ajout d'un membre.
Revue fonctionnelle:
pas de revue fonctionnelle, le test sera un test de perf en prod.
Guillaume Gouin
Cédric Damioli
Votes:
0Vote for this issue
Watchers:
4Start watching this issue
Created:
Updated:
Resolved:
{"errorMessages":["jqlTooComplex"],"errors":{}}
[{"id":-1,"name":"My open issues","jql":"assignee = currentUser() AND resolution = Unresolved order by updated DESC","isSystem":true,"sharePermissions":[],"requiresLogin":true},{"id":-2,"name":"Reported by me","jql":"reporter = currentUser() order by created DESC","isSystem":true,"sharePermissions":[],"requiresLogin":true},{"id":-4,"name":"All issues","jql":"order by created DESC","isSystem":true,"sharePermissions":[],"requiresLogin":false},{"id":-5,"name":"Open issues","jql":"resolution = Unresolved order by priority DESC,updated DESC","isSystem":true,"sharePermissions":[],"requiresLogin":false},{"id":-9,"name":"Done issues","jql":"statusCategory = Done order by updated DESC","isSystem":true,"sharePermissions":[],"requiresLogin":false},{"id":-3,"name":"Viewed recently","jql":"issuekey in issueHistory() order by lastViewed DESC","isSystem":true,"sharePermissions":[],"requiresLogin":false},{"id":-6,"name":"Created recently","jql":"created >= -1w order by created DESC","isSystem":true,"sharePermissions":[],"requiresLogin":false},{"id":-7,"name":"Resolved recently","jql":"resolutiondate >= -1w order by updated DESC","isSystem":true,"sharePermissions":[],"requiresLogin":false},{"id":-8,"name":"Updated recently","jql":"updated >= -1w order by updated DESC","isSystem":true,"sharePermissions":[],"requiresLogin":false}]
Revue technique:
La PR a 3 commit:
Revue fonctionnelle:
pas de revue fonctionnelle, le test sera un test de perf en prod.