kumarpritam863 commented on PR #19341:
URL: https://github.com/apache/kafka/pull/19341#issuecomment-2987117286

   Hi @mimaison ,
   Thanks for the input — I completely agree it's worth discussing this 
carefully.
   
   That said, I'm a bit unsure how this would break existing deployments, since 
client.id is primarily used for logging, metrics, and monitoring. It doesn’t 
affect Kafka's core delivery semantics like group.id or partition assignment. 
From what I understand, the only change here would be in how the client.id 
appears in logs and metrics.
   
   Also, similar changes have been made in the past — for example, we updated 
the client ID format for consumers of the management topic from 
<cluster>--statuses to <cluster>-statuses, and a few versions ago we modified 
the default client ID of internal sink consumers.
   
   That said, I agree it's best to bring this to the dev mailing list. I'll 
send an email shortly summarizing the context and asking whether this requires 
a KIP.
   
   Thanks again!


-- 
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: jira-unsubscr...@kafka.apache.org

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

Reply via email to