Karthik Kambatla commented on YARN-2241:

I ll have to retract that +1. The fix is good, but the test doesn't do much. 

Actually, the test doesn't fail without the fix. Is that intentional? Without 
this patch, these exceptions are merely logged and not thrown. I am okay with a 
patch without the test since we are just changing the logging.

> Show nicer messages when ZNodes already exist in ZKRMStateStore on startup
> --------------------------------------------------------------------------
>                 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
> 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