Hi,
I think what should be specified is also the way you get all active
sessions. If you want to broadcast messages (initiated by an other
service) you want to be able to say "getallActiveSessions().forEach(
s-> s.send("message"))". In case of failover this acitve sessions must
be synchronized between the nodes. Also a Session id is still missing.
If you want to notify one user the current demo way with
Map<User,Session> is not a good solution. When the Session has a unique
id you can store the relation across the deployment.
Thanks,
Andy