twitter/finagle

Don't mess with class loaders... or do it gently

Open

#357 ouverte le 1 avr. 2015

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)Scala (1 435 forks)batch import
good first issue

Métriques du dépôt

Stars
 (8 864 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Issue is described in following thread: https://groups.google.com/forum/#!topic/finaglers/snRvOYwljbI Basically Play's auto-reloading breaks Finagle's reflection based API. Also @jproper gives good explanation of what's going on and how to fix it here: https://github.com/playframework/playframework/issues/4159

Guide contributeur

Direction de recherche
Commencez par lire le fil de discussion Google Groups et le ticket Play #4159 référencés pour comprendre le contexte du problème et la correction proposée. Ensuite, examinez la base de code de Finagle pour les appels d'API basés sur la réflexion, en particulier dans le module core, afin de comprendre comment les chargeurs de classe sont utilisés. La correction implique probablement d'ajuster l'utilisation du chargeur de classe pour éviter les conflits avec le rechargement automatique de Play. Aucune PR active n'existe, une implémentation originale basée sur la discussion est donc nécessaire.
Stack technique
scala
Domaine
backendapi
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
4
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
3-5 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
JVM class loadingScala programmingFinagle internalsPlay auto reloading
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
20