sijie commented on a change in pull request #8261:
URL: https://github.com/apache/pulsar/pull/8261#discussion_r505651881



##########
File path: 
pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/TenantsBase.java
##########
@@ -105,6 +105,13 @@ public void createTenant(
 
         try {
             NamedEntity.checkName(tenant);
+            List<String> tenants = globalZk().getChildren(path(POLICIES), 
false);

Review comment:
       You shouldn't get the list of tenants from zookeeper first. You should 
check `maxTenants` first. If `maxTenants` is larger than 0, this feature is 
enabled. Then you get the list of tenants.




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

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


Reply via email to