twitter/finagle

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

Open

#357 geöffnet am 1. Apr. 2015

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Scala (1.435 Forks)batch import
good first issue

Repository-Metriken

Stars
 (8.864 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Beginnen Sie mit dem Lesen des referenzierten Google Groups Threads und des Play Issues #4159, um den Kontext des Problems und den vorgeschlagenen Fix zu verstehen. Untersuchen Sie dann die Codebasis von Finagle auf reflexionsbasierte API Aufrufe, insbesondere im Core Modul, um die Verwendung von Class Loadern zu verstehen. Der Fix beinhaltet wahrscheinlich eine Anpassung der Class Loader Nutzung, um Konflikte mit dem automatischen Neuladen von Play zu vermeiden. Es gibt keine aktiven PRs, daher ist eine eigene Implementierung basierend auf der Diskussion erforderlich.
Tech Stack
scala
Domain
backendapi
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
4
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
3-5 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
JVM class loadingScala programmingFinagle internalsPlay auto reloading
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
20