cache_decorator.py is the only file in the ./lib/gitiles directory that depends on AppEngine stuff, and the only thing that depends on cache_decorator.py is gitiles_repository.py. Ideally we should break that dependency (so that the gitiles library is independent of AppEngine) by offering a non-cached version of GitilesRepository (as part of the gitiles library), with the cached version as a subclass in some directory like ./services/appengine_utils
(Just to be clear, this ticket uses the terminology of https://codereview.chromium.org/2344443005/#ps240001)
cache_decorator.py is the only file in the ./lib directory that depends on AppEngine stuff, and the only thing that depends on cache_decorator.py is ./lib/gitiles/gitiles_repository.py. Ideally we should break that dependency (so that the gitiles library is independent of AppEngine) by offering a non-cached version of GitilesRepository (as part of the gitiles library), with the cached version as a subclass (in some other directory like ./services/appengine_utils or something).
(Just to be clear, this ticket uses the terminology of https://codereview.chromium.org/2344443005/#ps240001)
Comment 1 by wrengr@chromium.org
, Oct 28 2016