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

Cleanup worker creation for Spring Boot Auto Configuration and add validation of the explicit worker config #1308

Merged

Conversation

Spikhalskiy
Copy link
Contributor

@Spikhalskiy Spikhalskiy commented Jul 14, 2022

Cleans up and simplifies worker creation in Spring Boot Auto Configuration.
Add validation of the explicit worker config duplicate conflicts.
Avoids conflicts with auto-discovered implementations and adds detailed logging about the conflict resolution actions that were made.

Improving #8

@Spikhalskiy Spikhalskiy added this to the 1.15.0 milestone Jul 14, 2022
@Spikhalskiy Spikhalskiy force-pushed the spring-boot-cleanup-worker-creation branch from 902f4da to 75838f0 Compare July 14, 2022 22:51
@Spikhalskiy Spikhalskiy force-pushed the spring-boot-cleanup-worker-creation branch 2 times, most recently from 6e07530 to ba7ae1b Compare July 15, 2022 17:05
@Spikhalskiy Spikhalskiy marked this pull request as ready for review July 15, 2022 17:06
@Spikhalskiy Spikhalskiy force-pushed the spring-boot-cleanup-worker-creation branch 12 times, most recently from 135b583 to 15d40d9 Compare July 16, 2022 01:02
Copy link
Member

@cretz cretz left a comment

Choose a reason for hiding this comment

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

Looks good. Confirmed that 1) configuring two workers w/ the same name is a fail, and 2) auto-implementations share workers by task queue

@Spikhalskiy Spikhalskiy force-pushed the spring-boot-cleanup-worker-creation branch 5 times, most recently from e019e0c to 886e36c Compare July 18, 2022 18:27
@Spikhalskiy Spikhalskiy force-pushed the spring-boot-cleanup-worker-creation branch 9 times, most recently from 7c71567 to 3a3f0a7 Compare July 19, 2022 03:43
@Spikhalskiy Spikhalskiy force-pushed the spring-boot-cleanup-worker-creation branch from 3a3f0a7 to ab6aa75 Compare July 19, 2022 03:52
@Spikhalskiy Spikhalskiy merged commit 2917651 into temporalio:master Jul 19, 2022
@Spikhalskiy Spikhalskiy deleted the spring-boot-cleanup-worker-creation branch July 19, 2022 04:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants