[Predator] Add local cache for get command output.

appengine/findit/common/repo_util.py

Index: appengine/findit/common/repo_util.py
diff --git a/appengine/findit/common/repo_util.py b/appengine/findit/common/repo_util.py
index 511b54be3a7bfaa5675d6e1b1a98b709e234d66f..06e9ff5c83144e1517637ff3f0d7b7fbdb500fd7 100644
--- a/appengine/findit/common/repo_util.py
+++ b/appengine/findit/common/repo_util.py
@@ -8,6 +8,8 @@ import re
 import subprocess
 import urllib2
 
+from common import cache_decorator
+
 CODE_REVIEW_URL_PATTERN = re.compile(
     '^(?:Review URL|Review-Url): (.*\d+).*$', re.IGNORECASE)
 COMMIT_POSITION_PATTERN = re.compile(
@@ -76,7 +78,7 @@ def GetRevertedRevision(message):
       return reverted_revision_match.group(1)
 
 
-# TODO(katesonia): Add local cache for this function.
+@cache_decorator.Cached('Command-output', cacher=cache_decorator.LocalCacher())

[Martin Barbella, 2016/10/26 23:21:20]

Is the format used here for the namespace consistent with the other uses of
Cached? I don't particularly like this format, but I'm fine with it so long as
it's consistent with the rest of the usage.

[Sharu Jiang, 2016/10/27 21:59:05]

The other 2 are 'Gitiles-json-view' for SendRequestForJsonResponse and
'Gitiles-text-view' for SendRequestForTextResponse. I think they are consistent
or do you have any better name?

 def GetCommandOutput(command):
   """Gets the output stream of executable command.