Updated Branches: refs/heads/1.5.1-SNAPSHOT 59680b32d -> b85c8b5c1
ACCUMULO-2250 Move error logging in Fate.transitionToFailed to beginning of method Signed-off-by: Eric Newton <eric.new...@gmail.com> Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/b85c8b5c Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/b85c8b5c Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/b85c8b5c Branch: refs/heads/1.5.1-SNAPSHOT Commit: b85c8b5c1fcd05c79ac777fc9f2e4085dcbb1b71 Parents: 59680b3 Author: Vikram Srivastava <vikr...@cloudera.com> Authored: Fri Jan 24 01:38:44 2014 -0800 Committer: Eric Newton <eric.new...@gmail.com> Committed: Fri Jan 24 10:38:42 2014 -0500 ---------------------------------------------------------------------- fate/src/main/java/org/apache/accumulo/fate/Fate.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/accumulo/blob/b85c8b5c/fate/src/main/java/org/apache/accumulo/fate/Fate.java ---------------------------------------------------------------------- diff --git a/fate/src/main/java/org/apache/accumulo/fate/Fate.java b/fate/src/main/java/org/apache/accumulo/fate/Fate.java index bd36edb..670154e 100644 --- a/fate/src/main/java/org/apache/accumulo/fate/Fate.java +++ b/fate/src/main/java/org/apache/accumulo/fate/Fate.java @@ -96,9 +96,11 @@ public class Fate<T> { } private void transitionToFailed(long tid, Repo<T> op, Exception e) { + String tidStr = String.format("%016x", tid); + log.warn("Failed to execute Repo, tid=" + tidStr, e); store.setProperty(tid, EXCEPTION_PROP, e); store.setStatus(tid, TStatus.FAILED_IN_PROGRESS); - log.warn("Failed to execute Repo, tid=" + String.format("%016x", tid), e); + log.info("Updated status for Repo with tid=" + tidStr + " to FAILED_IN_PROGRESS"); } private void processFailed(long tid, Repo<T> op) {
