akka/akka-core

Serialization issues with Option

Ouverte

#29 923 ouverte le 30 déc. 2020

Voir sur GitHubGuide contributeur
 (3 commentaires) (0 réaction) (0 personne assignée)Scala (3 547 forks)batch import
1 - triagedhelp wanted

Métriques du dépôt

Stars
 (13 277 étoiles)
Métriques de merge PR
 (Merge moyen 8j 19h) (10 PRs mergées en 30 j)

Description

Some example projects and also the current style guide recommend using Option for state in event sourced behaviors (entities). This causes two issues:

  1. Double serialization - Option is caught by MiscMessageSerializer, which delegates payload serialization to another serializer based on configuration and then serializes the result again with ContainerFormats.Option.
  2. Payload serialization is always synchronous, so this results in blocking IO in a case where state uses AsyncSerializer.

These issues could be fixed by implementing following changes:

  1. Altering documentation to suggest using sealed trait hierarchies or case classes which contain Option to express empty state.
  2. Converting MiscMessageSerializer to AsyncSerializer.

Behavior seems obviously undesirable, so I'm wondering if I am missing something or does this actually need fixing?

I've noticed this issue with Lagom 1.6.4 entities, but since they fallback to Akka actors for everything I assume this is a general Akka issue.

Guide contributeur

Direction de recherche
Examinez comment MiscMessageSerializer gère les types Option, en particulier la double sérialisation et le blocage synchrone. Examinez la configuration de la sérialisation et comment AsyncSerializer est utilisé.
Stack technique
scala
Domaine
backend
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.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
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
ScalaAkkaSerialization
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40