-
Notifications
You must be signed in to change notification settings - Fork 152
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
SpringBoot - Add registered workflow and activity impl info to workers template #1986
Conversation
Signed-off-by: Tihomir Surdilovic <[email protected]>
return this; | ||
} | ||
|
||
public List<String> getRegisteredActivityInfo() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you think there is other information users may want us to expose in the future about activities or workflows other then their name?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
May make sense to return a list of objects so we can add new fields if decided to expose more info.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok good point, will update
...gure-alpha/src/main/java/io/temporal/spring/boot/autoconfigure/template/WorkersTemplate.java
Show resolved
Hide resolved
Signed-off-by: Tihomir Surdilovic <[email protected]>
Signed-off-by: Tihomir Surdilovic <[email protected]>
@Quinn-With-Two-Ns made updates, let me know if ok now |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we mark this api as @Experimental
Signed-off-by: Tihomir Surdilovic <[email protected]>
Added Experimental flag to getRegisteredInfo method |
Update to WorkersTemplate to expose registered workflow and activity class info per task queue.
Allows users to build custom actuator endpoints that display this.
Sample that uses this info to create a custom actuator endpoint temporalio/samples-java#576