akka/akka-http

Java DSL methods on ws.Message should be hidden from Scala

Ouverte

#927 ouverte le 5 mars 2017

Voir sur GitHubGuide contributeur
 (1 commentaire) (0 réaction) (0 personne assignée)Scala (598 forks)batch import
1 - triagedhelp wantedt:coret:java

Métriques du dépôt

Stars
 (1 311 étoiles)
Métriques de merge PR
 (Merge moyen 1j 10h) (2 PRs mergées en 30 j)

Description

The Scala DSL requires pattern matching to deal with TextMessage vs. BinaryMessage. Calling the content-assisted options leads to the Java side of things with mystifying error messages and lots of head scratching (e.g. folding over the stream with Scala 2.11 and being told that Function2 is expected where I am clearly supplying a 2-ary lambda expression).

The Java DSL methods are just misleading and should not show up on the Scala type. This can be achieved by making them private[akka], which translates to public for Java and is binary compatible.

It might be desirable to either document this better or add methods for the Scala DSL as well.

Guide contributeur

Direction de recherche
Examinez la classe ws.Message dans Akka HTTP pour identifier les méthodes Java DSL qui doivent être masquées de Scala. Étudiez comment appliquer le modificateur d'accès `private[akka]` à ces méthodes tout en maintenant la compatibilité binaire. Vérifiez les tests existants pour ws.Message pour garantir que les modifications ne cassent rien. Envisagez d'ajouter des méthodes DSL Scala si nécessaire.
Stack technique
scalajava
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
ScalaAkka HTTP
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65