-
Notifications
You must be signed in to change notification settings - Fork 44
Update Dead-Letter Queue Code samples #287
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
Conversation
gklijs
left a comment
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 👍
smcvb
left a comment
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.
Couple of remarks here.
Add AF 4.6.1 release notes
Add Multi-Tenancy extesions 4.6.1 release notes
Add Tracing extesions 4.6.1 release notes
Add JGroups and MultiTenancy minor release link to summary
…ingEventProcessors (cherry picked from commit 19e4ba7)
workerExecutor (cherry picked from commit 84913ee)
Append RevisionSnapshotFilter description to the Filtering Snapshot Events section. This should further clarify that all a user has to do to enable revision-based snapshot filtering, is add the @revision annotation. #enhancement/revision-snapshot-filter
Add a notice stating why Sagas do not support the DLQ. #enhancement/saga-dlq-notice
Introduce a dedicated token stealing section onto to the Streaming Processor page. The intent is to guide users further in what token stealing is, why it happens, when it happens, and what the consequence may be to them. #288
Add missing link to UnitOfWork #288
Replace command sample for third-party service, as command dispatching would only occur again in a distributed scenario. Although defining this wouldn't be off, it's out of the scope of the intent of the enumartion. #288
Co-authored-by: Sara Torrey <sara.torrey@axoniq.io>
Append `RevisionSnapshotFilter` description to Filtering Snapshot Events sub-section
Co-authored-by: Sara Torrey <sara.torrey@axoniq.io>
Co-authored-by: Sara Torrey <sara.torrey@axoniq.io>
Co-authored-by: Sara Torrey <sara.torrey@axoniq.io>
Co-authored-by: Sara Torrey <sara.torrey@axoniq.io>
Co-authored-by: Sara Torrey <sara.torrey@axoniq.io>
Co-authored-by: Sara Torrey <sara.torrey@axoniq.io>
Replace ascertain for guarantee for simplicity in language use. #288
Add 4.6.2 release notes
Add notices on Dead-Letter Queue support for Sagas
[#288] Add Token Stealing subsection to Tracking Tokens
Fine tune code samples for DLQ configuration #287
smcvb
left a comment
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.
My concerns have been addressed, hence I'm approving this pull request.
Closes AxonFramework/AxonFramework#2394