Skip to content

feat(batch): implement batch processing, controllers and tests#1106

Merged
tom-rm-meyer-ISST merged 10 commits intofeat/1103-partner-update-batchfrom
feat/batch-controller
Mar 9, 2026
Merged

feat(batch): implement batch processing, controllers and tests#1106
tom-rm-meyer-ISST merged 10 commits intofeat/1103-partner-update-batchfrom
feat/batch-controller

Conversation

@tom-rm-meyer-ISST
Copy link
Copy Markdown
Contributor

@tom-rm-meyer-ISST tom-rm-meyer-ISST commented Mar 6, 2026

Description

solves #1108

Implements the

  • Services: PartnerDataUpdateBatchProcessService
  • Controller

Docs via openApi

Tests

  • Controller
  • Services for Batch Processing of PartnerDataUpdate

Pre-review checks

Please ensure to do as many of the following checks as possible, before asking for committer review:

  • DEPENDENCIES are up-to-date. Dash license tool. Committers can open IP issues for restricted libs.
  • Copyright and license header are present on all affected files (TRG 7.02
  • Documentation Notice are present on all affected files (TRG 7.07)
  • If helm chart has been changed, the chart version has been bumped to either next major, minor or patch level (compared to released chart).
  • Changelog updated (changelog.md) with PR reference and brief summary.
  • Frontend version bumped, if needed (frontend/package.json, frontend/package-lock.json)
  • Backend version bumped, if needed (backend/pom.xml)
  • Open API specification updated, if controllers have been changed (use python script scripts/generate_openapi_yaml.py with running customer backend)

@tom-rm-meyer-ISST tom-rm-meyer-ISST force-pushed the feat/batch-controller branch 2 times, most recently from e4628ad to e12b05a Compare March 6, 2026 15:21
@tom-rm-meyer-ISST tom-rm-meyer-ISST marked this pull request as ready for review March 6, 2026 16:15
@tom-rm-meyer-ISST tom-rm-meyer-ISST changed the title Feat/batch controller feat(batch): implement batch processing, controllers and tests Mar 6, 2026
Copy link
Copy Markdown
Member

@ReneSchroederLJ ReneSchroederLJ left a comment

Choose a reason for hiding this comment

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

Very solid work overall. Just some edge cases for your consideration and some minor findings.

Comment thread docs/architecture/06_runtime_view.md Outdated
Copy link
Copy Markdown
Member

@ReneSchroederLJ ReneSchroederLJ left a comment

Choose a reason for hiding this comment

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

LGTM

@tom-rm-meyer-ISST tom-rm-meyer-ISST merged commit fca3765 into feat/1103-partner-update-batch Mar 9, 2026
11 checks passed
@tom-rm-meyer-ISST tom-rm-meyer-ISST deleted the feat/batch-controller branch March 9, 2026 06:53
tom-rm-meyer-ISST added a commit that referenced this pull request Mar 9, 2026
* feat(batch): prepare backend (#1105)

* feat(batch): prepare backend

* chore: update changelog

* fix: incorporate review

* chore: add license header

* feat(batch): implement batch processing, controllers and tests (#1106)

* feat(batch): prepare backend

* chore: update changelog

* fix: incorporate review

* chore: add license header

* feat(batch): add controllers and tests

* chore: update changelog

* fix(PartnerDataUpdateBatchProcessServiceTest): switch mixed direction for DoS

* fix: incorporate changes from othter pr

* chore: cleanup and review incorp

* feat(batch): implement frontend, cleanup scheduler and documentation (#1107)

* feat(batch): prepare backend

* chore: update changelog

* fix: incorporate review

* chore: add license header

* feat(batch): add controllers and tests

* chore: update changelog

* fix(PartnerDataUpdateBatchProcessServiceTest): switch mixed direction for DoS

* fix: incorporate changes from othter pr

* feat(batch): add view to frontend

* fix: incorporate changes from batch

* feat: added retention scheduler

* docs(batch): document changes

* feat(batch): add sorting for batch runs

* chore: cleanup and review incorp

* refactor(BatchView): improve type safety

Co-authored-by: René Schröder <131770181+ReneSchroederLJ@users.noreply.github.com>

* fix: improve typization and fix pagination

* fix(PartnerDataUpdateBatchProcessServiceImpl): prevent parallel runs

* chore: bump vulnerabilities in frontend

---------

Co-authored-by: René Schröder <131770181+ReneSchroederLJ@users.noreply.github.com>

* docs(Migration_Guide.md): added duplicate exception note for operational data post endoints

* fix(values.yaml): removed linting issue trailing spaces

---------

Co-authored-by: René Schröder <131770181+ReneSchroederLJ@users.noreply.github.com>
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.

[Story] Provide Controllers and Test for Update Partner Data Batch Feature

3 participants