Skip to content
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

Multitenant support #7

Merged
merged 30 commits into from Sep 15, 2022
Merged

Multitenant support #7

merged 30 commits into from Sep 15, 2022

Conversation

schananas
Copy link
Contributor

Adds Multi-tenant support for this extension

@smcvb smcvb requested review from a team, gklijs and YvonneCeelie and removed request for a team September 12, 2022 15:18
…-support

# Conflicts:
#	multitenancy-spring-boot-autoconfigure/pom.xml
#	multitenancy-spring-boot-autoconfigure/src/main/java/org/axonframework/extensions/multitenancy/autoconfig/AxonServerTenantProvider.java
#	multitenancy-spring-boot-autoconfigure/src/main/java/org/axonframework/extensions/multitenancy/autoconfig/ContextObject.java
#	multitenancy-spring-boot-autoconfigure/src/main/java/org/axonframework/extensions/multitenancy/autoconfig/MultiTenancyAutoConfiguration.java
#	multitenancy-spring-boot-autoconfigure/src/main/java/org/axonframework/extensions/multitenancy/autoconfig/MultiTenancyAxonServerAutoConfiguration.java
#	multitenancy-spring-boot-autoconfigure/src/main/java/org/axonframework/extensions/multitenancy/autoconfig/MultiTenantDataSourceManager.java
#	multitenancy-spring-boot-autoconfigure/src/main/java/org/axonframework/extensions/multitenancy/autoconfig/MultiTenantEventProcessorControlService.java
#	multitenancy-spring-boot-autoconfigure/src/main/java/org/axonframework/extensions/multitenancy/autoconfig/TenantConfiguration.java
#	multitenancy-spring-boot-autoconfigure/src/main/java/org/axonframework/extensions/multitenancy/autoconfig/TenantCorrelationProvider.java
#	multitenancy-spring-boot-starter/pom.xml
#	multitenancy/pom.xml
#	multitenancy/src/main/java/org/axonframework/extensions/multitenancy/TenantWrappedTransactionManager.java
#	multitenancy/src/main/java/org/axonframework/extensions/multitenancy/components/MultiTenantAwareComponent.java
#	multitenancy/src/main/java/org/axonframework/extensions/multitenancy/components/NoSuchTenantException.java
#	multitenancy/src/main/java/org/axonframework/extensions/multitenancy/components/TargetTenantResolver.java
#	multitenancy/src/main/java/org/axonframework/extensions/multitenancy/components/TenantConnectPredicate.java
#	multitenancy/src/main/java/org/axonframework/extensions/multitenancy/components/TenantDescriptor.java
#	multitenancy/src/main/java/org/axonframework/extensions/multitenancy/components/TenantProvider.java
#	multitenancy/src/main/java/org/axonframework/extensions/multitenancy/components/commandhandeling/MultiTenantCommandBus.java
#	multitenancy/src/main/java/org/axonframework/extensions/multitenancy/components/commandhandeling/TenantCommandSegmentFactory.java
#	multitenancy/src/main/java/org/axonframework/extensions/multitenancy/components/eventhandeling/MultiTenantEventProcessor.java
#	multitenancy/src/main/java/org/axonframework/extensions/multitenancy/components/eventhandeling/TenantEventProcessorSegmentFactory.java
#	multitenancy/src/main/java/org/axonframework/extensions/multitenancy/components/eventstore/MultiTenantEventStore.java
#	multitenancy/src/main/java/org/axonframework/extensions/multitenancy/components/eventstore/TenantEventSegmentFactory.java
#	multitenancy/src/main/java/org/axonframework/extensions/multitenancy/components/queryhandeling/MultiTenantQueryBus.java
#	multitenancy/src/main/java/org/axonframework/extensions/multitenancy/components/queryhandeling/MultiTenantQueryUpdateEmitter.java
#	multitenancy/src/main/java/org/axonframework/extensions/multitenancy/components/queryhandeling/TenantQuerySegmentFactory.java
#	multitenancy/src/main/java/org/axonframework/extensions/multitenancy/components/queryhandeling/TenantQueryUpdateEmitterSegmentFactory.java
#	multitenancy/src/main/java/org/axonframework/extensions/multitenancy/configuration/MultiTenantEventProcessingModule.java
- Change inception year to current year
- Move tests from starter module to the intended module
- Make MultiTenantEventStore#storeSnapshot public
- Move mockito dependency to right pom file
- Update junit jupiter to 5.9.0
- Use https for license file reference
- Update jacoco maven plugin to 0.8.8
- Update assertj to 3.23.1
Ensure the Spring Boot starter POM works as intended, following the Axon
 Framework starter pom, by:
- Setting the parent to spring-boot-starters
- Setting groupId, artifactId and version number in the main file
- Adding a name, for clarity
- Adding organization information
- Adding license information
- Adding issue management information
- Adding the release-sign-artifacts profile
- Adding build information
- Adding distribution management information
- Adding scm information
The auto-config test did not include the new
AxonTracingAutoConfiguration class. Due to this, the Axon Framework
components failed to retrieve a SpanFactory, as none was created.
Switch dependency to spring-boot-autoconfigure, as that's what a starter
 pom should do.
@smcvb smcvb self-assigned this Sep 15, 2022
Copy link
Member

@smcvb smcvb left a comment

Choose a reason for hiding this comment

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

Went through this pull request myself to make some needed fixes. In all, everything seems fine and is tested within sample applications sufficiently. As such, I approve of this PR.

@smcvb smcvb merged commit 503f165 into master Sep 15, 2022
@smcvb smcvb deleted the feature/multitenant-support branch September 15, 2022 12:06
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.

None yet

2 participants