Skip to content

Migrate to Axon Framework 5 - #456

Open
smcvb wants to merge 2 commits into
mainfrom
af5-migration
Open

Migrate to Axon Framework 5#456
smcvb wants to merge 2 commits into
mainfrom
af5-migration

Conversation

@smcvb

@smcvb smcvb commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

This pull request migrates the entirety of this project to Axon Framework 5.
The rewrite was done through our OpenRewrite script, in combination with the skill we have for migration (which can be found here).

Additionally, some modules have been dropped entirely from this project, in favor of the dedicated examples directory that's now part of Axon Framework.
The modules that have a replacement there are:

  1. Multitenancy
  2. Set-based validation
  3. Upcasting / Message Transformation

Lastly, the Saga module has been removed entirely, in favor of the Workflow module that's already been added to this repository.

Rewrite everything to AF5, using the OpenRewrite script. Furthermore,
some modules have been dropped entirely in favor of the examples modules
 in Axon Framework itself. Those that have been dropped are documented
 as such.
Upgrade to workflows 0.2.0
@MateuszNaKodach

Copy link
Copy Markdown
Contributor

Do you perhaps have any findings (or could you ask the Agent) on how we could improve our migration skills after this work?

Comment thread multitenancy/README.md
@@ -1,54 +0,0 @@
# Multi-tenant playground

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you remove that?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because it's a part of Axon Framework now! I'd expect samples to move over eventually. So, I documented that they've moved.

@@ -1,39 +0,0 @@
# Set Based Consistency Validation

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you remove the whole sample?

* @author Yvonne Ceelie
*/
@SuppressWarnings("unused")
public abstract class EventUpcasterChainFactory {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also I don't get it why you removed this... or if it's not useful, because we have sample in the axon framework repository itself, I'm not sure if we need ANY of examples in this repository.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because it's a part of Axon Framework now! I'd expect samples to move over eventually. So, I documented that they've moved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants