You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@camel.apache.org by GitBox <gi...@apache.org> on 2021/07/14 06:33:26 UTC

[GitHub] [camel-quarkus] jamesnetherton commented on issue #2901: Cannot re-process old Kafka messages because of Quarkus overriding the group.id

jamesnetherton commented on issue #2901:
URL: https://github.com/apache/camel-quarkus/issues/2901#issuecomment-879631426


   Sorry about this one, it's my bad. The map `put` should of course be `putIfAbsent`. I think we can make the properties merge behaviour configurable based on whether the Quarkus k8s service binding capability is present. And if it is, only then do we set up the custom `QuarkusKafkaClientFactory`.
   
   I'll also look into tweaking the Quarkus Kafka extension, as I don't see the point of it adding the `group.id` config, even when no other Kafka configuration has been provided. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org