YARN-2132. ZKRMStateStore.ZKAction#runWithRetries doesn't log the exception it encounters. (Vamsee Yarlagadda via kasha)

git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/branches/branch-2@1601066 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Karthik Kambatla 2014-06-07 06:13:22 +00:00
parent 3cc1e9fc31
commit d5379b29a2
2 changed files with 6 additions and 3 deletions

View File

@ -127,6 +127,9 @@ Release 2.5.0 - UNRELEASED
YARN-2122. In AllocationFileLoaderService, the reloadThread should be created YARN-2122. In AllocationFileLoaderService, the reloadThread should be created
in init() and started in start(). (Robert Kanter via kasha) in init() and started in start(). (Robert Kanter via kasha)
YARN-2132. ZKRMStateStore.ZKAction#runWithRetries doesn't log the exception
it encounters. (Vamsee Yarlagadda via kasha)
OPTIMIZATIONS OPTIMIZATIONS
BUG FIXES BUG FIXES

View File

@ -998,13 +998,13 @@ public class ZKRMStateStore extends RMStateStore {
throw new StoreFencedException(); throw new StoreFencedException();
} }
} catch (KeeperException ke) { } catch (KeeperException ke) {
LOG.info("Exception while executing a ZK operation.", ke);
if (shouldRetry(ke.code()) && ++retry < numRetries) { if (shouldRetry(ke.code()) && ++retry < numRetries) {
LOG.info("Waiting for zookeeper to be connected, retry no. + " LOG.info("Retrying operation on ZK. Retry no. " + retry);
+ retry);
Thread.sleep(zkRetryInterval); Thread.sleep(zkRetryInterval);
continue; continue;
} }
LOG.debug("Error while doing ZK operation.", ke); LOG.info("Maxed out ZK retries. Giving up!");
throw ke; throw ke;
} }
} }