From cca44ac36a0c483895447ef989d55920c359da8f Mon Sep 17 00:00:00 2001 From: Felipe Monteiro Date: Fri, 29 Jun 2018 18:31:33 -0400 Subject: [PATCH] optimization: Remove needless json.loads from middleware This patchset removes a needless json.loads from middleware which only adds needless overhead to Deckhand. Change-Id: I0e2a329c25198e80b42ca8b821baf6032509eec7 --- deckhand/control/middleware.py | 6 ------ 1 file changed, 6 deletions(-) diff --git a/deckhand/control/middleware.py b/deckhand/control/middleware.py index e5140bf7..cea31fcd 100644 --- a/deckhand/control/middleware.py +++ b/deckhand/control/middleware.py @@ -17,7 +17,6 @@ import yaml import falcon from oslo_config import cfg from oslo_log import log as logging -from oslo_serialization import jsonutils as json import six import deckhand.context @@ -170,11 +169,6 @@ class YAMLTranslator(HookableMiddlewareMixin, object): resp_attr = getattr(resp, attr) - try: - resp_attr = json.loads(resp_attr) - except (TypeError, ValueError): - pass - if isinstance(resp_attr, dict): setattr(resp, attr, yaml.safe_dump(resp_attr)) elif isinstance(resp_attr, (list, tuple)):