pjfanning commented on issue #2579:
URL: https://github.com/apache/pekko/issues/2579#issuecomment-3650915394

   @hanishi I know that DurableStateBehavior has API hooks for signals but to 
me, it appears that EventSourced needs the signals because side effects need to 
handled after recovery. DurableState doesn't appear to have the same issue. I 
am not an expert in persistence but I'm approaching this from the perspective 
that if the signals were important in DurableState, they would be better 
documented.
   
   
https://pekko.apache.org/docs/pekko/current/typed/persistence.html#event-handler
   > Side effects should be performed in thenRun from the [command 
handler](https://pekko.apache.org/docs/pekko/current/typed/persistence.html#command-handler)
 after persisting the event or from the 
[RecoveryCompleted](https://pekko.apache.org/japi/pekko/snapshot/org/apache/pekko/persistence/typed/RecoveryCompleted.html)
 after 
[Recovery](https://pekko.apache.org/docs/pekko/current/typed/persistence.html#recovery).
   
   
https://pekko.apache.org/docs/pekko/current/typed/durable-state/persistence.html#effects-and-side-effects
   
   The Durable State docs have good detail about handling side effects but do 
not mention signals at all. I think this might be deliberate.
   
   @ptrdom @samueleresca @He-Pin @raboof @mdedetrich may be able to provide a 
better insight than me.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to