Skip to content

[Fix #1380] Adding AllStrategyCorrelationInfo customization.#1381

Merged
fjtirado merged 1 commit into
serverlessworkflow:mainfrom
fjtirado:Fix_#1380
May 11, 2026
Merged

[Fix #1380] Adding AllStrategyCorrelationInfo customization.#1381
fjtirado merged 1 commit into
serverlessworkflow:mainfrom
fjtirado:Fix_#1380

Conversation

@fjtirado
Copy link
Copy Markdown
Collaborator

@fjtirado fjtirado commented May 11, 2026

Fix #1380

Copilot AI review requested due to automatic review settings May 11, 2026 14:35
@fjtirado fjtirado marked this pull request as draft May 11, 2026 14:42
@fjtirado fjtirado force-pushed the Fix_#1380 branch 2 times, most recently from 41b8437 to 79387de Compare May 11, 2026 14:50
Copilot AI reviewed May 11, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@fjtirado fjtirado marked this pull request as ready for review May 11, 2026 15:02
Copilot AI review requested due to automatic review settings May 11, 2026 15:02
Copilot AI reviewed May 11, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copilot AI review requested due to automatic review settings May 11, 2026 15:38
@fjtirado fjtirado force-pushed the Fix_#1380 branch 2 times, most recently from 0204703 to 664e0a2 Compare May 11, 2026 15:39
Copilot AI reviewed May 11, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 4 comments.

Comment thread ...e/src/main/java/io/serverlessworkflow/impl/scheduler/InMemoryAllStrategyCorrelationInfo.java
Comment thread impl/core/src/main/java/io/serverlessworkflow/impl/WorkflowApplication.java Outdated
Comment thread impl/test/src/test/java/io/serverlessworkflow/impl/test/ScheduleEventConsumerTest.java Outdated
Comment thread impl/test/src/test/java/io/serverlessworkflow/impl/test/ScheduleEventConsumerTest.java
Copilot AI review requested due to automatic review settings May 11, 2026 15:48
Copilot AI reviewed May 11, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Comment thread impl/core/src/main/java/io/serverlessworkflow/impl/WorkflowApplication.java
Comment thread ...e/src/main/java/io/serverlessworkflow/impl/scheduler/InMemoryAllStrategyCorrelationInfo.java
Comment thread impl/core/src/main/java/io/serverlessworkflow/impl/scheduler/ScheduledEventConsumer.java
@fjtirado
[Fix serverlessworkflow#1380] Adding AllStrategyCorrelationInfo custo…
9342118 
…mization.

Signed-off-by: fjtirado <ftirados@redhat.com>
@fjtirado fjtirado merged commit ded4a30 into serverlessworkflow:main May 11, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Feat: Allow support of Schedule.on on multi JVM env

2 participants

@fjtirado