-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[MODINVOSTO-187] Implement audit outbox pattern for sending kafka events about invoice updates #198
Conversation
2f764a9
to
21581e3
Compare
a02a551
to
b9294dd
Compare
94de983
to
303396e
Compare
be4b880
to
a2244e6
Compare
0e2486d
to
73a93aa
Compare
73a93aa
to
b419790
Compare
@@ -172,6 +184,35 @@ | |||
<groupId>org.apache.logging.log4j</groupId> | |||
<artifactId>log4j-slf4j-impl</artifactId> | |||
</dependency> | |||
<dependency> | |||
<groupId>org.apache.kafka</groupId> | |||
<artifactId>kafka-clients</artifactId> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
these 2 dependenices are exist in folio-kafka-wrapper: kafka-clients, kafka-junit
If possible we can use them from this shared library
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Both are necessary as kafka-junit in needed to startup embedded cluster, and without kafka-clients the tests fail to work properly with kafka
Quality Gate passedIssues Measures |
Purpose
[MODINVOSTO-187] Implement audit outbox pattern for sending kafka events about invoice updates
[MODINVOSTO-188] Implement audit outbox pattern for sending kafka events about invoice line updates
Approach
Verification:
After enabling module, the new tables are created:
After creating invoice and invoice line the outbox events are stored:
Sending logs to kafka:
Integration tests: