Re: [PR] [bugfix]Add logs to track sequence of events for table creation [pinot]

["snleee (via GitHub)" <gi...@apache.org>]

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: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org