From 172556ff034102de42c4c5d2db9e15ac6b66922f Mon Sep 17 00:00:00 2001
From: Dmitry <dmitry.petrov@nevesomo.com>
Date: Sun, 26 Nov 2017 03:20:38 -0800
Subject: [PATCH] Read target metric file, not state file

---
 dvc/command/show_workflow.py |  2 --
 dvc/git_wrapper.py           | 21 ++-------------------
 2 files changed, 2 insertions(+), 21 deletions(-)

diff --git a/dvc/command/show_workflow.py b/dvc/command/show_workflow.py
index 4d9d794b1..8c0d2585a 100644
--- a/dvc/command/show_workflow.py
+++ b/dvc/command/show_workflow.py
@@ -1,5 +1,3 @@
-import os
-
 from dvc.command.common.base import CmdBase
 from dvc.logger import Logger
 
diff --git a/dvc/git_wrapper.py b/dvc/git_wrapper.py
index 18ef1c7c0..5230da715 100644
--- a/dvc/git_wrapper.py
+++ b/dvc/git_wrapper.py
@@ -2,6 +2,7 @@ import os
 import re
 from collections import defaultdict
 
+from dvc import utils
 from dvc.exceptions import DvcException
 from dvc.logger import Logger
 from dvc.config import Config, ConfigI
@@ -379,30 +380,12 @@ class GitWrapper(GitWrapperI):
         if target not in changed_files:
             return False, None
 
-        metric = self._read_metric_from_state_file(hash, target, settings)
+        metric = utils.parse_target_metric_file(target)
         if metric is None:
             return False, None
 
         return True, metric
 
-    def _read_metric_from_state_file(self, hash, target, settings):
-        try:
-            data_item = settings.path_factory.data_item(target)
-        except DataItemError as ex:
-            Logger.warn('Target file {} is not data item: {}'.format(target, ex))
-            return None
-
-        try:
-            cmd_corresponded_state_file = ['git', 'show', '{}:{}'.format(hash, data_item.state.relative)]
-            state_file_content = Executor.exec_cmd_only_success(cmd_corresponded_state_file)
-        except ExecutorError as ex:
-            msg = '[dvc-git] Cannot obtain content of target symbolic file {} with hash {}: {}'
-            Logger.warn(msg.format(target, hash, ex))
-            return None
-
-        state_file = StateFile.loads(state_file_content, settings)
-        return state_file.single_target_metric
-
     @staticmethod
     def get_merges_map():
         # git log --merges --all --abbrev=7