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/trunk@1601065 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Karthik Kambatla 2014-06-07 06:12:59 +00:00
parent fb7ea6d8bc
commit 9958072d53
2 changed files with 6 additions and 3 deletions

View File

@ -142,6 +142,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;
} }
} }