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