Yunyung commented on PR #20203:
URL: https://github.com/apache/kafka/pull/20203#issuecomment-3475876821

   > > LGTM. Just need to update the KIP afterward.
   > 
   > sure. thanks for your view. already done yesterday and send mail to 
comminuty.
   
   I see. BrokerReadyCallback is not actually a public interface that will be 
exposed to users. It may be better to move it from the `Public Interface` 
section to `Proposed Changes`.
   Furthermore, it is good to elaborate on the behavior change for 
TopicBasedRemoteLogMetadataManager in the `Public Interface` section.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to