Aggarwal-Raghav commented on code in PR #473:
URL: https://github.com/apache/tez/pull/473#discussion_r3037107978
##########
tez-api/src/main/java/org/apache/tez/client/registry/zookeeper/ZkConfig.java:
##########
@@ -54,27 +55,42 @@ public ZkConfig(Configuration conf) {
zkQuorum = conf.get(TezConfiguration.TEZ_AM_ZOOKEEPER_QUORUM);
Preconditions.checkArgument(!Strings.isNullOrEmpty(zkQuorum), "zkQuorum
cannot be null or empty");
- String fullZkNamespace = ZK_NAMESPACE_PREFIX;
+ String fullZkAMNamespace = ZK_NAMESPACE_PREFIX;
- String namespace = conf.get(TezConfiguration.TEZ_AM_REGISTRY_NAMESPACE,
+ String amNamespace = conf.get(TezConfiguration.TEZ_AM_REGISTRY_NAMESPACE,
TezConfiguration.TEZ_AM_REGISTRY_NAMESPACE_DEFAULT);
- Preconditions.checkArgument(!Strings.isNullOrEmpty(namespace), "namespace
cannot be null or empty");
+ Preconditions.checkArgument(!Strings.isNullOrEmpty(amNamespace),
"namespace cannot be null or empty");
- fullZkNamespace = appendNamespace(fullZkNamespace, namespace);
+ fullZkAMNamespace = appendNamespace(fullZkAMNamespace, amNamespace);
+
+ String fullZkTaskNameSpace = ZK_NAMESPACE_PREFIX;
Review Comment:
TezChild will write its path in zk, _the multiple container is because of
multiple restart for tez child in docker compose, please ignore it_
<img width="1508" height="276" alt="Image"
src="https://github.com/user-attachments/assets/33f3b70b-889f-47ca-a484-cd023de37cd8";
/>
--
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]
