getappmap / getappmap/appmap-java
ConcurrentAccessException on CodeObject.children
Personne n'a encore pris cette issue.
- Langage dominant
- Java
- Étoiles
- 88
- Forks
- 17
- Merge moyen
- 2 j 2 h
- PR mergées (30 j)
- 1
Description
Hmm. Perhaps this is the cause, and if it is I believe the call site can be synchronized to avoid making CodeObject/CodeObjectTree thread safe:
https://github.com/applandinc/appmap-java/blob/less-mem_20200817/src/main/java/com/appland/appmap/record/Recorder.java#L110-L113
In short, writeEvent used to be synchronized. It's no longer synchronized due to the fact that we can encounter a lock when calling toString into the client application. It's certainly worth looking into. I'll defer to you @apotterri if you'd like to look into it now or as a follow on item.
Originally posted by @dustinbyrne in https://github.com/applandinc/appmap-java/pull/29#discussion_r475896188
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par Recorder.java au site d’appel de writeEvent référencé aux lignes 110-113, puis examinez CodeObject.children et CodeObjectTree pour le chemin d’accès concurrent signalé. Reproduisez ou confirmez d’une autre manière si un writeEvent non synchronisé provoque ConcurrentAccessException, et établissez une résolution vérifiée pour cet échec.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- java
- Domaine
- devtools
- Type d'issue
- Bug
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- À clarifier
- Accessibilité débutants
- 30/100