Skip to content
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

Kafka config override #2406

Merged
merged 2 commits into from
Dec 12, 2023
Merged

Conversation

ozangunalp
Copy link
Collaborator

Allow creating nested connector configs for configuring inner clients/channels. Useful for Kafka clients created for Kafka DLQ and Delayed Retry

cescoffier
cescoffier previously approved these changes Dec 8, 2023
@@ -133,7 +133,7 @@
<!-- Ensure log4j1 log backend with slf4j -->
<dependency>
<groupId>org.slf4j</groupId>
<artifactId>slf4j-log4j12</artifactId>
<artifactId>slf4j-reload4j</artifactId>
<scope>test</scope>
Copy link
Contributor

Choose a reason for hiding this comment

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

What's the reason?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

If I recall correctly before we changed the whole log4j12 dependency to reload4j which did not work for the quickstart samples. But I realize that we don't have any test Kafka/pulsar logs without the recent versions of reload4j.

@cescoffier cescoffier merged commit f277c46 into smallrye:main Dec 12, 2023
4 checks passed
@ozangunalp ozangunalp added this to the 4.13.0 milestone Dec 15, 2023
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.

2 participants