fix(docs/migration-paths): document @Namespace as the @ProcessingGroup replacement#4639
Open
MateuszNaKodach wants to merge 1 commit into
Open
fix(docs/migration-paths): document @Namespace as the @ProcessingGroup replacement#4639MateuszNaKodach wants to merge 1 commit into
MateuszNaKodach wants to merge 1 commit into
Conversation
MateuszNaKodach
requested review from
hatzlj,
jangalinski and
smcvb
and removed request for
a team
MateuszNaKodach
added
Priority 1: Must
…replacement The projectors and event processors migration path claimed that a "namespace" replacement for @processinggroup was "expected to be introduced soon". It already exists as @namespace (org.axonframework.messaging.core.annotation.Namespace). Rework the section to reflect the current state: * show the AF4 to AF5 annotation and import swap, preserving the value * note @namespace's broader placement (class, package-info, module-info) * warn that the namespace value is a binding contract that must match the event processor name everywhere it is referenced, since a mismatch silently misroutes events with no compile-time signal * split processor assignment into Spring Boot and declarative (non-Spring) tabs, clarifying that annotation-driven assignment is a Spring Boot feature while non-Spring configurations register handlers explicitly
MateuszNaKodach
force-pushed
the
docs/namespace-replaces-processing-group
branch
from
370c8f6 to
ac911b4
Compare
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The "Projectors and Event Processors" migration path stated that a "namespace" replacement for
@ProcessingGroupwas "expected to be introduced soon." That replacement already exists:@Namespace(org.axonframework.messaging.core.annotation.Namespace). This PR rewrites that section to document the current state.Changes
Single file:
docs/reference-guide/modules/migration/pages/paths/projectors-event-processors.adoc@ProcessingGroup" → "Replacing@ProcessingGroupwith@Namespace".@Namespace's broader placement (class, enclosing class,package-info.java,module-info.java).MessagingConfigurer.eventProcessing(...).Verification
@Namespacesemantics against the framework code: Spring extension (DefaultProcessorModuleFactory,EventHandlerSelector) reads it for processor assignment; core (AnnotationMessageTypeResolver) reads it only for message-type namespace. Non-Spring core config does not use it for assignment.