gianm commented on issue #8715: "group.id" of ConsumerPropertie be replaced by "kafka-supervisor-%s" in kafka-indexing-service URL: https://github.com/apache/incubator-druid/issues/8715#issuecomment-545029347 Hi @Elvis0607, just wondering, why are you wanting to customize the consumer `group.id`? The design of Druid's Kafka indexing doesn't use Kafka consumer groups (Druid manages offsets itself). So the thinking was that it shouldn't matter what the `group.id` was set to. If it matters for some reason, then yeah, switching those lines would be useful. Or maybe splitting the properties such that a user can override some but not others (for example, overriding `enable.auto.commit` would I think always be a bad idea).
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
