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

Command Processing SPI #523

Open
wants to merge 61 commits into
base: master
Choose a base branch
from
Open

Conversation

m1l4n54v1c
Copy link
Contributor

A conversation starter for the new Command Processing SPI.

schananas and others added 15 commits October 26, 2022 16:41
…into command-processing-refactoring

# Conflicts:
#	axonserver-command-processing/src/main/java/io/axoniq/axonserver/commandprocesing/imp/CommandSubscriptionCache.java
return command response value returned by the interceptor
# Conflicts:
#	axonserver/src/main/java/io/axoniq/axonserver/message/command/CommandCache.java
#	axonserver/src/main/java/io/axoniq/axonserver/message/command/CommandDispatcher.java
#	axonserver/src/main/java/io/axoniq/axonserver/message/command/CommandHealthIndicator.java
#	axonserver/src/test/java/io/axoniq/axonserver/config/SystemInfoProviderTest.java
#	axonserver/src/test/java/io/axoniq/axonserver/message/command/CommandCacheTest.java
#	axonserver/src/test/java/io/axoniq/axonserver/message/command/CommandDispatcherTest.java
@MGathier MGathier marked this pull request as ready for review November 14, 2022 15:33
@MGathier MGathier added this to the Release 2023.0.0 milestone Nov 14, 2022
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 2, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 19 Code Smells

66.5% 66.5% Coverage
0.6% 0.6% Duplication

1 similar comment
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 19 Code Smells

66.5% 66.5% Coverage
0.6% 0.6% Duplication

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 19 Code Smells

66.6% 66.6% Coverage
0.6% 0.6% Duplication

MGathier added 5 commits June 6, 2023 12:03
# Conflicts:
#	axonserver-commons/pom.xml
#	axonserver-commons/src/main/java/io/axoniq/axonserver/api/common.kt
#	axonserver/src/main/java/io/axoniq/axonserver/config/AxonServerStandardConfiguration.java
#	axonserver/src/main/java/io/axoniq/axonserver/message/command/CommandMetricsWebSocket.java
#	axonserver/src/main/java/io/axoniq/axonserver/rest/MetricsRestController.java
#	axonserver/src/main/java/io/axoniq/axonserver/rest/PublicRestController.java
#	axonserver/src/main/java/io/axoniq/axonserver/transport/grpc/GrpcAuthentication.java
#	axonserver/src/main/java/io/axoniq/axonserver/transport/rest/PrincipalAuthentication.java
#	axonserver/src/test/java/io/axoniq/axonserver/rest/MetricsRestControllerTest.java
#	pom.xml
explicitely synchronize requests for the queue
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 18 Code Smells

66.2% 66.2% Coverage
0.6% 0.6% Duplication

@saratry saratry removed their assignment Jul 19, 2024
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.

4 participants