cserwen commented on code in PR #4523:
URL: https://github.com/apache/rocketmq/pull/4523#discussion_r912691981
##########
namesrv/src/main/java/org/apache/rocketmq/namesrv/routeinfo/RouteInfoManager.java:
##########
@@ -156,6 +156,10 @@ public RegisterBrokerResult registerBroker(
BrokerData brokerData = this.brokerAddrTable.get(brokerName);
if (null == brokerData) {
+ if (topicConfigWrapper.getTopicConfigTable().size() == 1) {
Review Comment:
> Are you primarily trying to solve the problem of not being able to create
a topic before first registering the broker
I describe the current problem in detail in the issue. If someone creates a
topic before the broker first registers with a new namesrv, the broker's first
registration will fail. The registration can only be completed when the data
version of TopicConfigManager is changed.
--
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]
