)]}'
{"oslo_messaging/_drivers/impl_rabbit.py":[{"author":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"change_message_id":"142ebde2b9c06a1965cf8e3bb68d86144c329076","unresolved":false,"context_lines":[{"line_number":1131,"context_line":"    # List of notification queue declared on the channel to avoid"},{"line_number":1132,"context_line":"    # unnecessary redeclaration. This list is resetted each time"},{"line_number":1133,"context_line":"    # the connection is resetted in Connection._set_current_channel"},{"line_number":1134,"context_line":"    PUBLISHER_DECLARED_QUEUES \u003d collections.defaultdict(set)"},{"line_number":1135,"context_line":""},{"line_number":1136,"context_line":"    def _publish_and_creates_default_queue(self, exchange, msg,"},{"line_number":1137,"context_line":"                                           routing_key\u003dNone, timeout\u003dNone):"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a122d0e_4b81e48e","side":"PARENT","line":1134,"updated":"2016-05-04 12:57:08.000000000","message":"I can\u0027t remember why this was a class attributes instead of a instance attribute...","commit_id":"7f4826737d6fe995e57a05e7c6210ba7538a7db4"}],"oslo_messaging/tests/drivers/test_impl_rabbit.py":[{"author":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"change_message_id":"142ebde2b9c06a1965cf8e3bb68d86144c329076","unresolved":false,"context_lines":[{"line_number":276,"context_line":"                try_send(e_active)"},{"line_number":277,"context_line":"                # Ensure it creates it"},{"line_number":278,"context_line":"                try_send(e_passive)"},{"line_number":279,"context_line":""},{"line_number":280,"context_line":"                with mock.patch(\u0027kombu.messaging.Producer\u0027, side_effect\u003dexc):"},{"line_number":281,"context_line":"                    # Should reset the cache and ensures the exchange does"},{"line_number":282,"context_line":"                    # not exists"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a122d0e_4bdba41b","side":"PARENT","line":279,"updated":"2016-05-04 12:57:08.000000000","message":"I think we should keep this test, perhaps by mocking Producer.publish instead of Producer.__init__","commit_id":"7f4826737d6fe995e57a05e7c6210ba7538a7db4"},{"author":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"change_message_id":"65015797a09cd9d86fac8afab0a4d369fa66ebfa","unresolved":false,"context_lines":[{"line_number":276,"context_line":"                try_send(e_active)"},{"line_number":277,"context_line":"                # Ensure it creates it"},{"line_number":278,"context_line":"                try_send(e_passive)"},{"line_number":279,"context_line":""},{"line_number":280,"context_line":"                with mock.patch(\u0027kombu.messaging.Producer\u0027, side_effect\u003dexc):"},{"line_number":281,"context_line":"                    # Should reset the cache and ensures the exchange does"},{"line_number":282,"context_line":"                    # not exists"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a122d0e_7ee8dc16","side":"PARENT","line":279,"in_reply_to":"1a122d0e_4bdba41b","updated":"2016-05-04 12:59:45.000000000","message":"Correction, gerrit haven\u0027t post my last comment version :( \n\nI think we should keep this test, perhaps by mocking exchange().declare() instead of kombu.messaging.Producer.\n\nWe need to ensure the cache cleared when we are disconnected from the broker.","commit_id":"7f4826737d6fe995e57a05e7c6210ba7538a7db4"}]}