)]}'
{"neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/impl_idl_ovn.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"0d6603f6bbe9990ca8a3339a4d9b3762b9ac1deb","unresolved":true,"context_lines":[{"line_number":693,"context_line":"                    ).execute(check_error\u003dTrue):"},{"line_number":694,"context_line":"                if not row.external_ids.get(\u0027port_id\u0027):"},{"line_number":695,"context_line":"                    ret_opts.append(self._format_dhcp_row(row))"},{"line_number":696,"context_line":"                    break"},{"line_number":697,"context_line":"        return ret_opts"},{"line_number":698,"context_line":""},{"line_number":699,"context_line":"    def get_all_dhcp_options(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"0a254fb5_26d3f98b","line":696,"updated":"2026-01-29 21:28:21.000000000","message":"Seems the above and below methods can be changed the same way?","commit_id":"a7f127392971b724b094c8d683f369d3c7dabffb"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"d436f75bf72c4b09ce85305672c6e7163544572d","unresolved":false,"context_lines":[{"line_number":693,"context_line":"                    ).execute(check_error\u003dTrue):"},{"line_number":694,"context_line":"                if not row.external_ids.get(\u0027port_id\u0027):"},{"line_number":695,"context_line":"                    ret_opts.append(self._format_dhcp_row(row))"},{"line_number":696,"context_line":"                    break"},{"line_number":697,"context_line":"        return ret_opts"},{"line_number":698,"context_line":""},{"line_number":699,"context_line":"    def get_all_dhcp_options(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"eeba3467_57d96b90","line":696,"in_reply_to":"0a254fb5_26d3f98b","updated":"2026-01-30 08:05:38.000000000","message":"I don\u0027t think `get_all_dhcp_options` can be optimized. But the upper method yes, I\u0027ll push a new PS.","commit_id":"a7f127392971b724b094c8d683f369d3c7dabffb"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"c99c38cc2d83fed01f81f2a579a8fc1dc601c681","unresolved":true,"context_lines":[{"line_number":672,"context_line":"    def get_subnet_dhcp_options(self, subnet_id, with_ports\u003dFalse):"},{"line_number":673,"context_line":"        subnet \u003d {}"},{"line_number":674,"context_line":"        ports \u003d []"},{"line_number":675,"context_line":"        for row in self.db_find_rows("},{"line_number":676,"context_line":"                \u0027DHCP_Options\u0027,"},{"line_number":677,"context_line":"                (\u0027external_ids\u0027, \u0027\u003d\u0027, {\u0027subnet_id\u0027: subnet_id})"},{"line_number":678,"context_line":"                ).execute(check_error\u003dTrue):"}],"source_content_type":"text/x-python","patch_set":2,"id":"7141657e_772243f2","line":675,"range":{"start_line":675,"start_character":24,"end_line":675,"end_character":36},"updated":"2026-02-02 10:01:42.000000000","message":"+1, good optimization","commit_id":"f6e0b6e47b76ff6a02d86ac47fd34d6286209007"}]}