HoustonPutman opened a new pull request, #510: URL: https://github.com/apache/curator/pull/510
This fix really just lets the `catch (KeeperException.NodeExistsException e)` clause catch exceptions thrown in the `if (createParentsIfNeeded)` clause. Previously, since the two `catch` statements were for the same try, the `KeeperException.NodeExistsException` logic would not be called if that error was thrown when creating the node after creating the parents. This case can easily be seen when trying to call create.orSetData() in parallel on the same node, while creating parents if needed. This is because the `NoNodeException` would be thrown for both parallel calls, but only one would be able to create the new node in the catch statement, and the other would get an un-catchable `NodeExistsException`. -- 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]
