Robert Kanter updated YARN-2241:

    Attachment: YARN-2241.patch

You're right, it doesn't fail without the fix; I must have checked it with 
something slightly different than the old code when I tried it.  In that case I 
don't think we need the test; it's a pretty simple fix and I was able to verify 
that it worked correctly.

I've uploaded a new patch that doesn't have the test.

> ZKRMStateStore: On startup, show nicer messages when znodes already exist
> -------------------------------------------------------------------------
>                 Key: YARN-2241
>                 URL: https://issues.apache.org/jira/browse/YARN-2241
>             Project: Hadoop YARN
>          Issue Type: Bug
>          Components: resourcemanager
>    Affects Versions: 2.5.0
>            Reporter: Robert Kanter
>            Assignee: Robert Kanter
>            Priority: Minor
>         Attachments: YARN-2241.patch, YARN-2241.patch
> When using the RMZKStateStore, if you restart the RM, you get a bunch of 
> stack traces with messages like 
> {{org.apache.zookeeper.KeeperException$NodeExistsException: KeeperErrorCode = 
> NodeExists for /rmstore}}.  This is expected as these nodes already exist 
> from before.  We should catch these and print nicer messages.

This message was sent by Atlassian JIRA

Reply via email to