jiafu1115 commented on PR #20007:
URL: https://github.com/apache/kafka/pull/20007#issuecomment-3004014985

   "we don't use another thread to initialize resources in 
TopicBasedRemoteLogMetadataManager [3] and throw exception immediately if there 
is an error."
   this is another good proposal. but I am worrying that if it will make the 
startup take more time so that the initial process fail due to potential loop 
for request for topic existed with connection
   
   So that is why I suppose to stop it when fail initial and still keep it in 
thread. Then we can delete this check in many methods:
    if (initializationFailed) { 
        // If initialization is failed, shutdown the broker. 
        throw new FatalExitError(); 
    } 
    
    WDTY?


-- 
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