International-Data-Spaces-Association / IDS-Messaging-Services

Latest: 7.0.0 - Lightweight and ready-to-use services to easily connect an IDS-Connector to different IDS-Infrastructure-Components.
Apache License 2.0
13 stars 6 forks source link

Update Infomodel and Serializer #634

Open SebastianOpriel opened 1 year ago

SebastianOpriel commented 1 year ago

There was a release for Infomodel Serializer with some performance improvements (v5.0.3). These changes could be integrated into messaging services, if no problems arise. The Infomodel itself got also updated. It may be discussed, if the Infomodel is backwards compatible and could be integrated smoothly into messaging services and then into DSC (but needs good compatibility checks). The reason is that IAIS is working on integrating messaging services into Broker Core for MDS see https://github.com/Mobility-Data-Space/MDS-Broker-Core/tree/feature/ids_messaging_services They would like to use the latest updates of serializer and infomodel dependencies in messaging-services

tmberthold commented 1 year ago

Would imply the following constellation for Messaging-Services (see linked PR):

From experience, there will also be breaking changes for the DSC and not just for the Messaging-Services itself, which will have to be tested and adressed in the run-up to the PR. So even if the PR in the Messaging-Services itself can be merged successfully, there could still be breaking conflicts in the DSC, which need to be evaluated beforehand before merging the linked PR.

tmberthold commented 1 year ago

There are two main breaking changes compared to the DSC:

  1. enum classes became interfaces with constants
  2. the newly introduced UriOrModelClass
SebastianOpriel commented 1 year ago

@rmsoyta could you provide some more feedback on requirements from your side? Maybe we could find a version which does not lead to incompabilities but would satisfy your requirements.

timwirtz86 commented 1 year ago

A short comment on that. Up to my current understanding, the breaking changes will be caused by the evolution of the version of the java classes (@tmberthold , @SebastianOpriel )? I think so because the serializer should be independent of the info model version used. It is included as test-dependency only and is (sadly) the sole target we test the serializer against. The changes in the interaction library (compared to the latest previous version) are marginal in the sense that we only updated the dependencies.

You could give it a trial and solely include the update of the serializer. This shouldn't cause breaking changes.

tmberthold commented 1 year ago

We have updated the serializer in the DSC as a test, but the e2e-tests of the DSC fail, so changes are hiding here as well. History of the failed test: https://github.com/International-Data-Spaces-Association/DataspaceConnector/actions/runs/3629932830/jobs/6123853530