Skip to content

Develop -> Master -> Main#587

Merged
ymo-sci merged 118 commits intomasterfrom
develop
Dec 29, 2025
Merged

Develop -> Master -> Main#587
ymo-sci merged 118 commits intomasterfrom
develop

Conversation

@ymo-sci
Copy link
Contributor

@ymo-sci ymo-sci commented Dec 29, 2025

Description

Merge develop to master, which will be renamed to main

Additional Notes

  • This PR fixes or works on following ticket(s): SIRI-1162
  • This PR is related to PR:

Checklist

  • Code change has been tested and works locally
  • Code was formatted via IntelliJ and follows SonarLint & best practices
  • Patch Tasks: Is local execution of Patch Tasks necessary? If so, please also mark the PR with the tag.

jakobvogel and others added 30 commits April 11, 2025 15:09
We have an areAllEmpty, but areAllFilled is not the same as !areAllEmpty

Fixes: OX-11912
Adds the Strings.areAllFilled
- fixes: DO-409
- fixes: DO-409
schedule will only work for default-branch so we must try it "life"

- fixes: DO-409
DO-409 First try for PR automatic tests with github-actions
…respecting i18n settings.

OX-11996
…as the `null` case is explicitly handled in the code, and some code locations in OX did not respect the constraint anyway.
idlira and others added 27 commits October 16, 2025 10:25
Fixes: SIRI-1146
Due to historic reasons, this class was placed in the xml package.
As it evolved into a full block generic HTTP client, we moved it to
the commons package.
By trying to re-use existing HttpClients, we can utilize the
internal connection pool and thus drastically speed up repeated
requests against the same host.

As a client might be customized through its builder, we now ask for
a clientSelector to distinguish these.

If null is used as clientSelector, no re-use will happen.
We now always prefer Duration in favor of ints. Also the
documentation was fixes as it was plain wrong.
This is breaking as some constants were public. Use HttpHeaders from
guava for proper constants.
Outcall.modifyClient(String) accepts a string to cache a
newly constructed HttpClient. null is used to signal that
the client must not be cached. To better document this,
an explicit constant was added so that

call.modifyClient(Outcall.CLIENT_SELECTOR_NO_CACHE)
sort of speaks for itself (in contrast to call.modifyClient(null)
Adds an explicit constant to better document behavior
By default, we check if the current TaskContext is active, but more complex scenarios might require additional checks.

Fixes: OX-12263
if something is submitted after the executor have been closed.

For cleanliness, empty the queue at the end.

Fixes: OX-12263
Improves the ParallelTaskExecutor
@ymo-sci ymo-sci added the 🛠️ Maintenance Translations, Code Cleanup, ... label Dec 29, 2025
@ymo-sci ymo-sci merged commit 545bd20 into master Dec 29, 2025
54 of 55 checks passed
@scireumskip scireumskip deleted the develop branch December 29, 2025 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🛠️ Maintenance Translations, Code Cleanup, ...

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants