From be224413bb0c53a4339c72f9c5c0c09687034ac3 Mon Sep 17 00:00:00 2001
From: Zhongjie Wu <www.zj30@gmail.com>
Date: Wed, 13 Jun 2012 14:22:26 -0700
Subject: [PATCH] changed checkout record code to make sure timeouts are
 recorded

---
 .../socket/clientrequest/ClientRequestExecutorPool.java | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

diff --git a/src/java/voldemort/store/socket/clientrequest/ClientRequestExecutorPool.java b/src/java/voldemort/store/socket/clientrequest/ClientRequestExecutorPool.java
index 6d015a2d5..1c7ee64bc 100644
--- a/src/java/voldemort/store/socket/clientrequest/ClientRequestExecutorPool.java
+++ b/src/java/voldemort/store/socket/clientrequest/ClientRequestExecutorPool.java
@@ -133,20 +133,21 @@ public class ClientRequestExecutorPool implements SocketStoreFactory {
      */
 
     public ClientRequestExecutor checkout(SocketDestination destination) {
+        // time checkout
+        long start = System.nanoTime();
+        ClientRequestExecutor clientRequestExecutor;
         try {
-            // time checkout
-            long start = System.nanoTime();
-            ClientRequestExecutor clientRequestExecutor = pool.checkout(destination);
+            clientRequestExecutor = pool.checkout(destination);
+        } catch(Exception e) {
+            throw new UnreachableStoreException("Failure while checking out socket for "
+                                                + destination + ": ", e);
+        } finally {
             long end = System.nanoTime();
             if(stats != null) {
                 stats.recordCheckoutTimeUs(destination, (end - start) / Time.NS_PER_US);
             }
-
-            return clientRequestExecutor;
-        } catch(Exception e) {
-            throw new UnreachableStoreException("Failure while checking out socket for "
-                                                + destination + ": ", e);
         }
+        return clientRequestExecutor;
     }
 
     /**
-- 
2.11.4.GIT