abhishekbafna commented on code in PR #15881:
URL: https://github.com/apache/pinot/pull/15881#discussion_r2120154611
##########
pinot-server/src/main/java/org/apache/pinot/server/starter/helix/HelixInstanceDataManager.java:
##########
@@ -84,6 +91,22 @@ public class HelixInstanceDataManager implements
InstanceDataManager {
private static final Logger LOGGER =
LoggerFactory.getLogger(HelixInstanceDataManager.class);
private final Map<String, TableDataManager> _tableDataManagerMap = new
ConcurrentHashMap<>();
+
+ // A cache for the logical table context
+ private static final String TABLE_CONFIG_PATH_PREFIX = "/CONFIGS/TABLE/";
Review Comment:
Defined them as below. Refactored their uses across the logical table code.
```
/**
* ZK paths used by Pinot.
*/
public static class ZkPaths {
public static final String LOGICAL_TABLE_PARENT_PATH = "/LOGICAL/TABLE";
public static final String LOGICAL_TABLE_PATH_PREFIX = "/LOGICAL/TABLE/";
}
```
--
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-unsubscr...@pinot.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
