Skip to content

Conversation

ricardozanini
Copy link
Member

Many thanks for submitting your Pull Request ❤️!

What this PR does / why we need it:
Fixes:

  1. Bump version.com.fasterxml.jackson from 2.19.2 to 2.20.0 #756
  2. Bump dev.langchain4j:langchain4j-agentic from 1.3.0-beta9 to 1.4.0-beta10 #752

Special notes for reviewers:

Additional information (if needed):

throw new IllegalArgumentException(
"ExecutorService is required for the ParallelAgentService");
}
// TODO: create an adapter or change or internal holder to accept a plain `Executor`.
Copy link
Member Author

Choose a reason for hiding this comment

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

@fjtirado do you think this is reasonable? Or do we must use an ExecutorService?

@@ -77,7 +77,8 @@

<!-- Dependencies versions, please keep in alphabetical order -->
<version.ch.qos.logback>1.5.18</version.ch.qos.logback>
<version.com.fasterxml.jackson>2.19.2</version.com.fasterxml.jackson>
<version.com.fasterxml.jackson>2.20.0</version.com.fasterxml.jackson>
<version.com.fasterxml.jackson.annotations>2.20</version.com.fasterxml.jackson.annotations>
Copy link
Member Author

Choose a reason for hiding this comment

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

I think this is a bug on their side, they released without the patch version number..

@ricardozanini ricardozanini merged commit 5d60c89 into serverlessworkflow:main Sep 2, 2025
3 checks passed
@ricardozanini ricardozanini deleted the upgrade-jackson-lc4j branch September 2, 2025 18:35
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.

1 participant