)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"41d4c3b82c8629199f603c394d4f688c6b5aafb9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f7b44dda_5c57d7cb","updated":"2026-06-24 13:48:20.000000000","message":"Nice catch, thanks! Just one syntactic recommendation in-line.","commit_id":"1edaaea32b8132576f3ab91ed6d7b535ea0175d6"},{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"168d96d9d6cb2d75804bcce1af922ef4625046be","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b983daed_46319324","updated":"2026-06-24 19:31:24.000000000","message":"lgtm, thanks!","commit_id":"b51e6cb04122541598af4e2cad766ac05ae32b46"}],"ovsdbapp/backend/ovs_idl/event.py":[{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"41d4c3b82c8629199f603c394d4f688c6b5aafb9","unresolved":true,"context_lines":[{"line_number":64,"context_line":"class RowEventHandler(ovsdb_event.RowEventHandler):"},{"line_number":65,"context_line":"    def notify(self, event, row, updates\u003dNone):"},{"line_number":66,"context_line":"        row \u003d idlutils.frozen_row(row)"},{"line_number":67,"context_line":"        super().notify(event, row, updates)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9c35ce9d_af86bd5a","side":"PARENT","line":67,"updated":"2026-06-24 13:48:20.000000000","message":"Maybe we could just add something on the parent class like\n```\ndef _on_matching(self, event, row, updates\u003dNone):\n    return row\n```\nand put the `if matching: row \u003d self._on_matching(event, row, updates)` in its `notify()` and then still use `super()` (or I guess even drop the `notify()` override altogether) and just override `_on_matching()` to avoid having to copy the method body.","commit_id":"10c1c984a138a78f29b468ad50bbd365fd247c2d"},{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"168d96d9d6cb2d75804bcce1af922ef4625046be","unresolved":false,"context_lines":[{"line_number":64,"context_line":"class RowEventHandler(ovsdb_event.RowEventHandler):"},{"line_number":65,"context_line":"    def notify(self, event, row, updates\u003dNone):"},{"line_number":66,"context_line":"        row \u003d idlutils.frozen_row(row)"},{"line_number":67,"context_line":"        super().notify(event, row, updates)"}],"source_content_type":"text/x-python","patch_set":1,"id":"4cb774d7_c4f72c39","side":"PARENT","line":67,"in_reply_to":"5f043154_bc3e17dc","updated":"2026-06-24 19:31:24.000000000","message":"Done","commit_id":"10c1c984a138a78f29b468ad50bbd365fd247c2d"},{"author":{"_account_id":39125,"name":"Felix Moebius","email":"felix.moebius@digits.schwarz","username":"felixmoebius"},"change_message_id":"3948f2fcd65dfc691acb687b42e05f52344e9a14","unresolved":true,"context_lines":[{"line_number":64,"context_line":"class RowEventHandler(ovsdb_event.RowEventHandler):"},{"line_number":65,"context_line":"    def notify(self, event, row, updates\u003dNone):"},{"line_number":66,"context_line":"        row \u003d idlutils.frozen_row(row)"},{"line_number":67,"context_line":"        super().notify(event, row, updates)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f043154_bc3e17dc","side":"PARENT","line":67,"in_reply_to":"9c35ce9d_af86bd5a","updated":"2026-06-24 17:49:58.000000000","message":"Agreed, thats certainly more elegant. Thanks!","commit_id":"10c1c984a138a78f29b468ad50bbd365fd247c2d"}]}