mynameborat commented on a change in pull request #1484:
URL: https://github.com/apache/samza/pull/1484#discussion_r606423160
##########
File path: samza-core/src/main/java/org/apache/samza/zk/ZkKeyBuilder.java
##########
@@ -27,6 +27,7 @@
* - /
* |- groupId/
* |- JobModelGeneration/
+ * |- activeJobModelVersion (data contains the most recent active
job model version)
Review comment:
I had this documented in the getter of the `ZkKeyBuilder` in the same
class. Here is the difference
```
Denotes the path where the most recent active job model version is stored.
The version of the job model is the
* most recent agreed upon version by the quorum. It differs from the
<i>jobModelVersion</i> path which may
* have a newer version of job model published by the leader in during
rebalance before consensus is achieved.
* @return the path where most recent active job model is stored
```
--
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]
