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

Event processor reset with admin api #193

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

trimoq
Copy link
Collaborator

@trimoq trimoq commented Jul 4, 2022

We added the new admin API as a third method to reset the tokens.
Further, we streamlined the Controller of the example to call the EventProcessorService for the selected method.
This allows to separate the things we want to show with the samples (using the three methods to reset the tokens) from the spring specific stuff like the rest controllers.

For the new features, the documentation has been updated, see also the ref-guide updates.

ToDo:

  • It remains to decide if we want to keep the spring specific controller stuff or if we remove that all-together and refer the users to a simple test. This would make playing with the example harder but understanding the relevant parts easier.

@trimoq trimoq force-pushed the feature/event-processor-reset-with-admin-api branch from 477ec78 to 195e5dc Compare July 19, 2022 08:54
@trimoq trimoq marked this pull request as ready for review July 19, 2022 08:54
@trimoq trimoq force-pushed the feature/event-processor-reset-with-admin-api branch 2 times, most recently from b2b0205 to d9a6fff Compare July 19, 2022 09:42
@trimoq trimoq force-pushed the feature/event-processor-reset-with-admin-api branch from d9a6fff to b51f8a9 Compare July 19, 2022 10:02
@trimoq trimoq requested a review from saratry December 7, 2022 10:27
@trimoq trimoq self-assigned this Dec 7, 2022
<dependencies>
<dependency>
<groupId>org.axonframework</groupId>
<artifactId>axon-spring-boot-starter</artifactId>
<version>4.5.14</version>
Copy link
Contributor

Choose a reason for hiding this comment

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

Why not the 4.8 form the bom?

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.

Code Sample and documentation for Distributed Event Processor Token Reset
4 participants