snleee commented on code in PR #11946:
URL: https://github.com/apache/pinot/pull/11946#discussion_r1381239913


##########
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/PinotHelixResourceManager.java:
##########
@@ -1573,6 +1575,8 @@ public void addTable(TableConfig tableConfig)
           + "If the external view is not removed after a long time, try 
restarting the servers showing up in the "
           + "external view");
     }
+
+    LOGGER.info("Adding table {}: Validate table tenant config", 
tableNameWithType);

Review Comment:
   Let's rephrase this as `Validate table configs`
   
   Also, let's add one line before 1594: `Creating table config in the property 
store`



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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to