-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Serialization reload on settings change #2730
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Serialization reload on settings change #2730
Conversation
Not sure if there is a better way how to handle this scenario. |
@zbynek001 which serializers do you have this problem for? User-defined ones? |
I've run into this with singleton & sharding. During startup, they are trying to inject default configs via |
4c9e017
to
7e5bda4
Compare
Looks good to me. We do the same thing for the Dispatchers subsystem as well. The change in itself is good. |
will check the failing test |
Should be fixed by #2952 |
reason was that two nodes (those with sharding) were using ClusterShardingMessageSerializer, and the Control node was using hyperion so it was not able to deserialize some messages |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me
Serializers configurations from extension default config are not correctly loaded.
At the time extension tries to inject it's default config with InjectTopLevelFallback, serializer configuration was already loaded and it's not reloaded afterwards. Currently only workaround I know about is providing all fallback configurations when creating ActorSystem itself. This pr tries to improve this.