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
Apply event sourcing to message publish processor #6376
Conversation
df249f6
to
c902242
Compare
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.
Thanks @saig0 I added some questions which would be cool if we discuss them before. Tbh it was a bit confusing for me that we introduced the CORRELATING for message subscritptions and have the CORRELATED for message start subscriptions 😅 and yes I have read the ZEP, but it doesn't make it more clearer.
protocol/src/main/java/io/zeebe/protocol/record/value/MessageRecordValue.java
Outdated
Show resolved
Hide resolved
engine/src/main/java/io/zeebe/engine/state/appliers/MessagePublishedApplier.java
Show resolved
Hide resolved
...main/java/io/zeebe/engine/state/appliers/MessageStartEventSubscriptionCorrelatedApplier.java
Show resolved
Hide resolved
messageState.put(message); | ||
|
||
// avoid correlating this message to the workflow again | ||
correlatingSubscriptions.visitBpmnProcessIds( |
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.
Where is this done now?
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.
I assume we write for all of them the follow up events?
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.
- insert message in state: https://github.com/zeebe-io/zeebe/blob/c9022428e0c0eb3c9f2df3d2395690a1e7bb3420/engine/src/main/java/io/zeebe/engine/state/appliers/MessagePublishedApplier.java#L38
- insert message correlation: https://github.com/zeebe-io/zeebe/blob/c9022428e0c0eb3c9f2df3d2395690a1e7bb3420/engine/src/main/java/io/zeebe/engine/state/appliers/MessageSubscriptionCorrelatingApplier.java#L48
- insert correlation for message start event: https://github.com/zeebe-io/zeebe/blob/c9022428e0c0eb3c9f2df3d2395690a1e7bb3420/engine/src/main/java/io/zeebe/engine/state/appliers/MessageStartEventSubscriptionCorrelatedApplier.java#L34
🛎️ @Zelldon do you have more questions? |
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.
thanks 👍
* use the state writer in the message publish processor to apply the state changes * extend the ES templates for the new record properties * disable the replay filter to restore the key generator for migrated processors
634c1af
to
e421bc4
Compare
bors r+ |
Build succeeded: |
Description
EventHandle
to share the common behavior to trigger start eventsdeadline
variables
workflowInstanceKey
,messageKey
,correlationKey
,variables
CORRELATING
CORRELATED
Hints for the reviewer:
Related issues
The last part of #6178.
closes #6178
Definition of Done
Not all items need to be done depending on the issue and the pull request.
Code changes:
backport stable/0.25
) to the PR, in case that fails you need to create backports manually.Testing:
Documentation: