From 1d66f4effa3e917b53b52c6ce3c3d5772cade962 Mon Sep 17 00:00:00 2001
From: birdstorm <samuelwyf@hotmail.com>
Date: Mon, 21 Jan 2019 18:37:55 +0800
Subject: [PATCH] Refine region refresh logic in RegionStoreClient.get (#27)

Signed-off-by: birdstorm <samuelwyf@hotmail.com>
---
 src/main/java/org/tikv/common/region/RegionStoreClient.java | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/main/java/org/tikv/common/region/RegionStoreClient.java b/src/main/java/org/tikv/common/region/RegionStoreClient.java
index ef026f5674..13d735c506 100644
--- a/src/main/java/org/tikv/common/region/RegionStoreClient.java
+++ b/src/main/java/org/tikv/common/region/RegionStoreClient.java
@@ -82,9 +82,6 @@ public class RegionStoreClient extends AbstractGRPCClient<TikvBlockingStub, Tikv
   // APIs for KV Scan/Put/Get/Delete
   public ByteString get(BackOffer backOffer, ByteString key, long version) {
     while (true) {
-      // we should refresh region
-      region = regionManager.getRegionByKey(key);
-
       Supplier<GetRequest> factory =
           () ->
               GetRequest.newBuilder()
@@ -105,6 +102,9 @@ public class RegionStoreClient extends AbstractGRPCClient<TikvBlockingStub, Tikv
       if (getHelper(backOffer, resp)) {
         return resp.getValue();
       }
+
+      // we should refresh region
+      region = regionManager.getRegionByKey(key);
     }
   }