)]}'
{"nova/objects/host_mapping.py":[{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"245f7f9ea0f18ecc9b7b3a2f791b71e01840f6d4","unresolved":false,"context_lines":[{"line_number":44,"context_line":"        }"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"    def _get_cell_mapping(self):"},{"line_number":47,"context_line":"        with db_api.api_context_manager.reader.using(_CONTEXT) as session:"},{"line_number":48,"context_line":"            cell_map \u003d (session.query(api_models.CellMapping)"},{"line_number":49,"context_line":"                        .join(api_models.HostMapping)"},{"line_number":50,"context_line":"                        .filter(api_models.HostMapping.host \u003d\u003d self.host)"}],"source_content_type":"text/x-python","patch_set":6,"id":"da85f559_17f05866","line":47,"updated":"2015-11-11 13:11:35.000000000","message":"this definitely needs commentary before landing this about why this is the requirement for getting to the second db, and the implications of the thread local context here. Both for reviewers and for posterity.","commit_id":"aa4de991c47141f0c3d3ab9a00f43ce6697819bb"},{"author":{"_account_id":10618,"name":"Pavel Kholkin","email":"p.v.holkin@mail.ru","username":"pkholkin"},"change_message_id":"c003e8a85fe4b7f76aca5600d6134a40040100de","unresolved":false,"context_lines":[{"line_number":44,"context_line":"        }"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"    def _get_cell_mapping(self):"},{"line_number":47,"context_line":"        with db_api.api_context_manager.reader.using(_CONTEXT) as session:"},{"line_number":48,"context_line":"            cell_map \u003d (session.query(api_models.CellMapping)"},{"line_number":49,"context_line":"                        .join(api_models.HostMapping)"},{"line_number":50,"context_line":"                        .filter(api_models.HostMapping.host \u003d\u003d self.host)"}],"source_content_type":"text/x-python","patch_set":6,"id":"da85f559_db6cb238","line":47,"in_reply_to":"da85f559_17f05866","updated":"2015-11-13 15:41:01.000000000","message":"yes, fixed, this should be the same context as everywhere","commit_id":"aa4de991c47141f0c3d3ab9a00f43ce6697819bb"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"d0c15b4bd50025c9ff111c91d725dbf4a05fb1f0","unresolved":false,"context_lines":[{"line_number":40,"context_line":"        }"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    def _get_cell_mapping(self):"},{"line_number":43,"context_line":"        with db_api.api_context_manager.reader.using(self._context) as session:"},{"line_number":44,"context_line":"            cell_map \u003d (session.query(api_models.CellMapping)"},{"line_number":45,"context_line":"                        .join(api_models.HostMapping)"},{"line_number":46,"context_line":"                        .filter(api_models.HostMapping.host \u003d\u003d self.host)"}],"source_content_type":"text/x-python","patch_set":7,"id":"7a5de9d1_a13ee988","line":43,"updated":"2016-01-30 09:36:50.000000000","message":"I would rather we just use the decorator here, really, and convert this to a static method.","commit_id":"7275f220292cc8eb3fdffcbfcd303fee2df37de0"}]}