)]}'
{"oslo_db/sqlalchemy/enginefacade.py":[{"author":{"_account_id":6849,"name":"Roman Podoliaka","email":"roman.podoliaka@gmail.com","username":"rpodolyaka"},"change_message_id":"d5eab0515befa091a029b95ffe851f4dd088da1f","unresolved":false,"context_lines":[{"line_number":145,"context_line":"            \u0027thread_checkin\u0027: _Default(True),"},{"line_number":146,"context_line":"            \u0027json_serializer\u0027: _Default(None),"},{"line_number":147,"context_line":"            \u0027json_deserializer\u0027: _Default(None),"},{"line_number":148,"context_line":"            \u0027logging_name\u0027: _Default(None)"},{"line_number":149,"context_line":"        }"},{"line_number":150,"context_line":"        self._maker_cfg \u003d {"},{"line_number":151,"context_line":"            \u0027expire_on_commit\u0027: _Default(False),"}],"source_content_type":"text/x-python","patch_set":11,"id":"bacf61ea_5b4dd598","line":148,"updated":"2016-08-01 16:58:05.000000000","message":"nit: this should probably go into a separate commit","commit_id":"8153b5652a87ffc5b48ed44d70f0c802cc000566"},{"author":{"_account_id":6849,"name":"Roman Podoliaka","email":"roman.podoliaka@gmail.com","username":"rpodolyaka"},"change_message_id":"d5eab0515befa091a029b95ffe851f4dd088da1f","unresolved":false,"context_lines":[{"line_number":317,"context_line":""},{"line_number":318,"context_line":"    def _maker_args_for_conf(self, conf):"},{"line_number":319,"context_line":"        maker_args \u003d self._args_for_conf(self._maker_cfg, conf)"},{"line_number":320,"context_line":"        maker_args[\u0027autocommit\u0027] \u003d maker_args.pop(\u0027__autocommit\u0027)"},{"line_number":321,"context_line":"        return maker_args"},{"line_number":322,"context_line":""},{"line_number":323,"context_line":"    def dispose_pool(self):"}],"source_content_type":"text/x-python","patch_set":11,"id":"bacf61ea_1b34fddb","line":320,"updated":"2016-08-01 16:58:05.000000000","message":"nit: this should probably go into a separate commit","commit_id":"8153b5652a87ffc5b48ed44d70f0c802cc000566"},{"author":{"_account_id":6849,"name":"Roman Podoliaka","email":"roman.podoliaka@gmail.com","username":"rpodolyaka"},"change_message_id":"d5eab0515befa091a029b95ffe851f4dd088da1f","unresolved":false,"context_lines":[{"line_number":419,"context_line":"        if apply_global:"},{"line_number":420,"context_line":"            self.apply_to(_context_manager)"},{"line_number":421,"context_line":""},{"line_number":422,"context_line":"    @classmethod"},{"line_number":423,"context_line":"    def apply_engine(self, engine, transaction_context_manager):"},{"line_number":424,"context_line":"        existing_factory \u003d transaction_context_manager._factory"},{"line_number":425,"context_line":"        maker \u003d existing_factory._writer_maker"}],"source_content_type":"text/x-python","patch_set":11,"id":"bacf61ea_7ba7f90a","line":422,"updated":"2016-08-01 16:58:05.000000000","message":"hmm, why classmethod here? Looks like an instance state is used below","commit_id":"8153b5652a87ffc5b48ed44d70f0c802cc000566"}]}