Jian He commented on YARN-3077:

[~chenchun],  thanks for working on this.
The newly added test is passing without the patch change. mind taking a deeper 
look ?
For test case: I suggest changing  ZK_RM_STATE_STORE_PARENT_PATH to use 
"/foo/bar" for the existing test cases, instead of adding a new unit test to 
test the until method.

> RM should create yarn.resourcemanager.zk-state-store.parent-path recursively
> ----------------------------------------------------------------------------
>                 Key: YARN-3077
>                 URL: https://issues.apache.org/jira/browse/YARN-3077
>             Project: Hadoop YARN
>          Issue Type: Improvement
>          Components: resourcemanager
>            Reporter: Chun Chen
>         Attachments: YARN-3077.patch
> If multiple clusters share a zookeeper cluster, users might use 
> /rmstore/${yarn.resourcemanager.cluster-id} as the state store path. If user 
> specified a customer value which is not a top-level path for 
> ${yarn.resourcemanager.zk-state-store.parent-path}, yarn should create parent 
> path first.

This message was sent by Atlassian JIRA

Reply via email to