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
[JBPM-9780] Accept all kafka properties #2510
Conversation
6a048cf
to
a0b718c
Compare
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.
Take into account that some properties have the same name for producer & consumer (CommonClientConfigs) and they should be added to both lists.
Also, a comment related to exception caught when getting the producer and a minor comment for enhancing the test.
...ervices-kafka/src/test/java/org/kie/server/services/jbpm/kafka/KafkaServerExtensionTest.java
Outdated
Show resolved
Hide resolved
...ver-services-kafka/src/main/java/org/kie/server/services/jbpm/kafka/KafkaServerProducer.java
Show resolved
Hide resolved
...er-services-kafka/src/main/java/org/kie/server/services/jbpm/kafka/KafkaServerExtension.java
Outdated
Show resolved
Hide resolved
c564789
to
d2fc196
Compare
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.
Looks good to me, excellent work @fjtirado !
Jenkins retest this please |
1 similar comment
Jenkins retest this please |
Kudos, SonarCloud Quality Gate passed! |
JIRA:
link