-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
No way to signal no change with ChangeEventHandler #32310
Comments
Alternatively, if we want to require that a |
Adding documentation makes sense. The event sequence number must be in sync with the durable state change revision, i.e. one event per change. Therefore I think it is best to not make the ChangeEvent optional (even though we could store that hidden ignore payload). Calling the change handler also for non-changes doesn't make sense, because we can't store additional events when there is no revision change. Again, sync of sequence numbers. |
From forum question: https://discuss.lightbend.com/t/changeeventhandler-no-op/10620/3
The
updateHandler
has signature(State, State, Command) => ChangeEvent
so there is no way to say a command had no effect on the state.The text was updated successfully, but these errors were encountered: