Return-Path: 
 <commits-return-8159-apmail-accumulo-commits-archive=accumulo.apache.org@accumulo.apache.org>
X-Original-To: apmail-accumulo-commits-archive@www.apache.org
Delivered-To: apmail-accumulo-commits-archive@www.apache.org
Received: from mail.apache.org (hermes.apache.org [140.211.11.3])
	by minotaur.apache.org (Postfix) with SMTP id 462FF10B09
	for <apmail-accumulo-commits-archive@www.apache.org>;
 Fri, 24 Jan 2014 15:34:05 +0000 (UTC)
Received: (qmail 28805 invoked by uid 500); 24 Jan 2014 15:34:03 -0000
Delivered-To: apmail-accumulo-commits-archive@accumulo.apache.org
Received: (qmail 28740 invoked by uid 500); 24 Jan 2014 15:34:02 -0000
Mailing-List: contact commits-help@accumulo.apache.org; run by ezmlm
Precedence: bulk
List-Help: <mailto:commits-help@accumulo.apache.org>
List-Unsubscribe: <mailto:commits-unsubscribe@accumulo.apache.org>
List-Post: <mailto:commits@accumulo.apache.org>
List-Id: <commits.accumulo.apache.org>
Reply-To: dev@accumulo.apache.org
Delivered-To: mailing list commits@accumulo.apache.org
Received: (qmail 28733 invoked by uid 99); 24 Jan 2014 15:34:02 -0000
Received: from tyr.zones.apache.org (HELO tyr.zones.apache.org)
 (140.211.11.114)
    by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 24 Jan 2014 15:34:02 +0000
Received: by tyr.zones.apache.org (Postfix, from userid 65534)
	id 0FD60904AAB; Fri, 24 Jan 2014 15:34:02 +0000 (UTC)
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
From: ecn@apache.org
To: commits@accumulo.apache.org
Message-Id: <1a840212e5fd44d2b83c3e6cb60c0f60@git.apache.org>
X-Mailer: ASF-Git Admin Mailer
Subject: git commit: ACCUMULO-2250 Move error logging in
 Fate.transitionToFailed to beginning of method
Date: Fri, 24 Jan 2014 15:34:02 +0000 (UTC)

Updated Branches:
  refs/heads/1.6.0-SNAPSHOT 11d803cf1 -> 930a6b84f


ACCUMULO-2250 Move error logging in Fate.transitionToFailed to beginning of method

Signed-off-by: Eric Newton <eric.newton@gmail.com>


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/930a6b84
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/930a6b84
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/930a6b84

Branch: refs/heads/1.6.0-SNAPSHOT
Commit: 930a6b84f84c87809bd6af0748358393b75b681f
Parents: 11d803c
Author: Vikram Srivastava <vikrams@cloudera.com>
Authored: Fri Jan 24 01:38:44 2014 -0800
Committer: Eric Newton <eric.newton@gmail.com>
Committed: Fri Jan 24 10:33:50 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/930a6b84/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 2d69647..3561fc8 100644
--- a/fate/src/main/java/org/apache/accumulo/fate/Fate.java
+++ b/fate/src/main/java/org/apache/accumulo/fate/Fate.java
@@ -99,9 +99,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) {