eclipse-basyx / basyx-java-server-sdk

MIT License
60 stars 47 forks source link

Add Apache Kafka Event Interface For Repositories #523

Open geso02 opened 2 weeks ago

geso02 commented 2 weeks ago

522

Description:

This pull request introduces an Apache Kafka event interface, enhancing message processing reliability and scalability, and making the system more suitable for larger and more demanding environments.

Key Features

Thank you for reviewing this addition!

aaronzi commented 2 weeks ago

Hi @geso02,

thank you very much for your PR and the nice addition to BaSyx. Can you please check the AAS Environment. There is a test that seems to be failing.

geso02 commented 1 week ago

I found some incorrect configurations in the upload within the AASEnvironment. The pathsToLoad used during bean creation was missing the @Value annotation in all places. I've now added it. In the authorization test case, we then encountered an issue because basyx.environment was now being referenced, but this test was not concerned with it and did not provide mocks for it, causing it to fail. I've now pushed the changes again and will check if there are issues in other areas as well.

geso02 commented 1 week ago

I’ve fixed the issue with the build by adjusting the registration of a Submodel that was intended for submodel-related tests but was mistakenly included in environment tests due to test dependencies. I’ve now ensured that the Submodel is only registered in the submodel tests using profiles, and it’s no longer active in the environment tests, which resolves the issue and allows the build to pass again.