)]}'
{"ceilometer/ipmi/platform/intel_node_manager.py":[{"author":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"change_message_id":"12f91dbbc10d669726855bc828d7e9ba2d3721e1","unresolved":false,"context_lines":[{"line_number":33,"context_line":"from ceilometer.openstack.common.gettextutils import _"},{"line_number":34,"context_line":"from oslo.config import cfg"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"import collections"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"def get_ordereddict(*args, **kwargs):"}],"source_content_type":"text/x-python","patch_set":1,"id":"baa201ad_9f5d9f4b","line":36,"updated":"2014-09-29 09:10:07.000000000","message":"how about moving this line before ln42?","commit_id":"770ef5b2473530652a210f46aca603b9b4fb20c2"},{"author":{"_account_id":7102,"name":"Thomas Bechtold","email":"thomas.bechtold@chainguard.dev","username":"toabctl"},"change_message_id":"39d054106deab2f2658c16cb65215b224ee850ee","unresolved":false,"context_lines":[{"line_number":33,"context_line":"from ceilometer.openstack.common.gettextutils import _"},{"line_number":34,"context_line":"from oslo.config import cfg"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"import collections"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"def get_ordereddict(*args, **kwargs):"}],"source_content_type":"text/x-python","patch_set":1,"id":"baa201ad_52cb1072","line":36,"in_reply_to":"baa201ad_9f5d9f4b","updated":"2014-09-30 05:55:15.000000000","message":"Done","commit_id":"770ef5b2473530652a210f46aca603b9b4fb20c2"},{"author":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"change_message_id":"12f91dbbc10d669726855bc828d7e9ba2d3721e1","unresolved":false,"context_lines":[{"line_number":42,"context_line":"        return collections.OrderedDict(*args, **kwargs)"},{"line_number":43,"context_line":"    except AttributeError:"},{"line_number":44,"context_line":"        import ordereddict"},{"line_number":45,"context_line":"        return ordereddict.OrderedDict(*args, **kwargs)"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"node_manager_init_retry \u003d cfg.IntOpt(\u0027node_manager_init_retry\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"baa201ad_ff022b0a","line":45,"updated":"2014-09-29 09:10:07.000000000","message":"how about we change the get_ordereddict() to return the class instance to avoid calling it every time instantiating a OrderedDict instance? e.g.\n\ndef get_ordereddict():\n    try:\n        import collections\n        return collections.OrderedDict\n    except AttributeError:\n        import ordereddict\n        return ordereddict.OrderedDict\n\nOrderedDict \u003d get_ordereddict()\n\n...\n\nxxx \u003d OrderedDict()","commit_id":"770ef5b2473530652a210f46aca603b9b4fb20c2"},{"author":{"_account_id":7102,"name":"Thomas Bechtold","email":"thomas.bechtold@chainguard.dev","username":"toabctl"},"change_message_id":"39d054106deab2f2658c16cb65215b224ee850ee","unresolved":false,"context_lines":[{"line_number":42,"context_line":"        return collections.OrderedDict(*args, **kwargs)"},{"line_number":43,"context_line":"    except AttributeError:"},{"line_number":44,"context_line":"        import ordereddict"},{"line_number":45,"context_line":"        return ordereddict.OrderedDict(*args, **kwargs)"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"node_manager_init_retry \u003d cfg.IntOpt(\u0027node_manager_init_retry\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"baa201ad_b288bc23","line":45,"in_reply_to":"baa201ad_ff022b0a","updated":"2014-09-30 05:55:15.000000000","message":"Done","commit_id":"770ef5b2473530652a210f46aca603b9b4fb20c2"}]}