)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"edbf7fd89429bf52c09c6596b8012bef32e5e647","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"48b3febf_c9591ca3","updated":"2026-08-17 16:09:19.000000000","message":"Should we add _sync_order to the base class here?","commit_id":"2463995c1cace4d688e1c8f38b95a0a26d8a8bfc"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"211c87f8a00c73e97c199b41d4a627f40698dc06","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1791120a_f40fbd62","in_reply_to":"14fe483d_c3c8499a","updated":"2026-08-17 18:58:49.000000000","message":"Right, it was only for the code to match the docs.","commit_id":"2463995c1cace4d688e1c8f38b95a0a26d8a8bfc"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"c7fd890bd44493a8f13c99109e599296c0a0d727","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5618c84b_cd41ac41","in_reply_to":"1791120a_f40fbd62","updated":"2026-08-17 19:03:23.000000000","message":"The docs were meant to show how to make a synchronizer that would be ordered after the default. Perhaps I should update the docs to make it clearer?","commit_id":"2463995c1cace4d688e1c8f38b95a0a26d8a8bfc"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"9d6ee0f63a5d04d06bface43e484db3f0f4bac9b","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"14fe483d_c3c8499a","in_reply_to":"48b3febf_c9591ca3","updated":"2026-08-17 16:15:50.000000000","message":"Only in sake of readability - this line https://review.opendev.org/c/openstack/neutron/+/1001147/1/neutron/cmd/ovn/neutron_ovn_db_sync_util.py#185 gets the class attribute but if it\u0027s not defined it defaults to 0. Thus we don\u0027t need to add it for functional purposes.","commit_id":"2463995c1cace4d688e1c8f38b95a0a26d8a8bfc"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"b43d7d2b9f108241c0191a66b354f7a9a3d9bdae","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"61a6574f_aed23d10","in_reply_to":"5618c84b_cd41ac41","updated":"2026-08-17 19:50:48.000000000","message":"No, I think it\u0027s fine, the example is for a non-core synchronizer. We\u0027ll see what others think.","commit_id":"2463995c1cace4d688e1c8f38b95a0a26d8a8bfc"}]}