)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"83b0c35c16c352ffdac960b753287dee9c179efb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1268315e_00877b53","updated":"2026-04-09 08:56:39.000000000","message":"I think it is ok to merge this one","commit_id":"027623ea07253aaea349478b884f8d2ede2f4619"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"b094aa9bcd931fb1729194d67bf5a2105e1c651a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ef5187b9_f96f6386","updated":"2026-04-10 08:02:50.000000000","message":"recheck failure is not related to the patch","commit_id":"027623ea07253aaea349478b884f8d2ede2f4619"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"304081ee920a20ab77777cd1c4cd4b35605252e1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5458400d_c9650bba","updated":"2026-04-09 18:44:42.000000000","message":"recheck https://review.opendev.org/c/openstack/designate-tempest-plugin/+/983844 is merged now","commit_id":"027623ea07253aaea349478b884f8d2ede2f4619"}],"designate/mdns/handler.py":[{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"0a44880260db4f312a17eced24183c79a1a26a84","unresolved":true,"context_lines":[{"line_number":321,"context_line":"            # zone name exists in multiple pools. The TSIG key on the"},{"line_number":322,"context_line":"            # request determines which pool\u0027s zone to serve, matching"},{"line_number":323,"context_line":"            # the approach used by _handle_axfr."},{"line_number":324,"context_line":"            zone \u003d None"},{"line_number":325,"context_line":"            try:"},{"line_number":326,"context_line":"                zone_criterion \u003d self._zone_criterion_from_request("},{"line_number":327,"context_line":"                    request, {\u0027name\u0027: name})"}],"source_content_type":"text/x-python","patch_set":1,"id":"a8a8c44e_ec04261a","line":324,"updated":"2026-03-29 10:04:01.000000000","message":"What is the impact on performance here?","commit_id":"027623ea07253aaea349478b884f8d2ede2f4619"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"94eceed9df1313cfaac6cf07fec2089de2d265df","unresolved":true,"context_lines":[{"line_number":321,"context_line":"            # zone name exists in multiple pools. The TSIG key on the"},{"line_number":322,"context_line":"            # request determines which pool\u0027s zone to serve, matching"},{"line_number":323,"context_line":"            # the approach used by _handle_axfr."},{"line_number":324,"context_line":"            zone \u003d None"},{"line_number":325,"context_line":"            try:"},{"line_number":326,"context_line":"                zone_criterion \u003d self._zone_criterion_from_request("},{"line_number":327,"context_line":"                    request, {\u0027name\u0027: name})"}],"source_content_type":"text/x-python","patch_set":1,"id":"c528690a_57e7232a","line":324,"in_reply_to":"a8a8c44e_ec04261a","updated":"2026-04-08 15:04:29.000000000","message":"For the common case (zone-apex queries like SOA), there\u0027s no extra cost — it\u0027s still 2 DB queries, just in reversed order. The zone-first lookup hits indexed columns (name + pool_id).\nFor subdomain queries, there\u0027s one additional DB call — the initial find_zone that raises ZoneNotFound before falling back to the recordset-first path.\nIn practice I think the impact should be negligible.","commit_id":"027623ea07253aaea349478b884f8d2ede2f4619"}]}