)]}'
{"oslo_messaging/_drivers/impl_rabbit.py":[{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"cbdd741d48f803dc8695621e9d4ddf3e72ee1ad8","unresolved":false,"context_lines":[{"line_number":888,"context_line":"        def _consume():"},{"line_number":889,"context_line":"            if self._new_consumers:"},{"line_number":890,"context_line":"                for tag, consumer in enumerate(self._consumers):"},{"line_number":891,"context_line":"                    if consumer in self._new_consumers:"},{"line_number":892,"context_line":"                        consumer.consume(tag\u003dtag)"},{"line_number":893,"context_line":"                self._new_consumers \u003d []"},{"line_number":894,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"1aca2d91_73deb312","line":891,"updated":"2015-05-07 14:47:20.000000000","message":"is this check so we don\u0027t redeclare an active consumer? i assume it\u0027s not possible that a consumer is in _new_consumer but not in _consumers?","commit_id":"1e17d83310d8aa7e8cf327c2da97046510fb27a0"},{"author":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"change_message_id":"55667501af072c1776d9d8022e29672f1860de8c","unresolved":false,"context_lines":[{"line_number":888,"context_line":"        def _consume():"},{"line_number":889,"context_line":"            if self._new_consumers:"},{"line_number":890,"context_line":"                for tag, consumer in enumerate(self._consumers):"},{"line_number":891,"context_line":"                    if consumer in self._new_consumers:"},{"line_number":892,"context_line":"                        consumer.consume(tag\u003dtag)"},{"line_number":893,"context_line":"                self._new_consumers \u003d []"},{"line_number":894,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"1aca2d91_2e58d231","line":891,"in_reply_to":"1aca2d91_73deb312","updated":"2015-05-07 14:58:24.000000000","message":"Yes, self._consumers is the complete list of consumers the one declared from amqp PoV and other. And self._new_consumers is the list of consumers not yet delcared from the amqp pov.\n\nself._consumers can only grow, when self._new_consumers is most of the times empty.","commit_id":"1e17d83310d8aa7e8cf327c2da97046510fb27a0"}]}