)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"ccb37380038a3595649a8378a9bbb405931a9522","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"83b80331_c093e2ca","updated":"2023-04-03 18:31:37.000000000","message":"I confirm the fix implements the feature.","commit_id":"9d92fe74d8a5875f2a9e9f1af3948186512db78b"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"66541401949e2917df297cb4b1e88779406714f5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"00a79302_46f911f1","updated":"2023-04-04 12:00:25.000000000","message":"looks ok","commit_id":"8e37d962ccccde31a335e131d16077e1a3db7636"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"83c3881f692d7a46cc8ced1a56bfac0cf7da45f9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"5571bac8_2433af50","updated":"2023-04-19 12:25:55.000000000","message":"Soft -1, just trying to understand the patch","commit_id":"831ae23f49a044b5bf6ab6f18ce08915be03fd3a"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"e179b3a9ef3e1d4755eb0d7def5c33407ef73cea","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"6d3adbeb_532a919e","updated":"2023-04-19 12:23:42.000000000","message":"Suggetions in line","commit_id":"831ae23f49a044b5bf6ab6f18ce08915be03fd3a"}],"ovsdbapp/api.py":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"e179b3a9ef3e1d4755eb0d7def5c33407ef73cea","unresolved":true,"context_lines":[{"line_number":190,"context_line":"        \"\"\""},{"line_number":191,"context_line":""},{"line_number":192,"context_line":"    @abc.abstractmethod"},{"line_number":193,"context_line":"    def db_clear(self, table, record, column, key\u003dNone):"},{"line_number":194,"context_line":"        \"\"\"Create a command to clear a field\u0027s value in a record"},{"line_number":195,"context_line":""},{"line_number":196,"context_line":"        :param table:  The OVS table containing the record to be modified"}],"source_content_type":"text/x-python","patch_set":4,"id":"faa44288_ab73aadd","line":193,"updated":"2023-04-19 12:23:42.000000000","message":"Should we make it \"keys\" (plural) and allow for clearing more than one key at a time ? Just like we could add multiple values with db_add()","commit_id":"831ae23f49a044b5bf6ab6f18ce08915be03fd3a"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"83c3881f692d7a46cc8ced1a56bfac0cf7da45f9","unresolved":true,"context_lines":[{"line_number":282,"context_line":"        :type record:     string"},{"line_number":283,"context_line":"        :param column:    The column whose value should be deleted"},{"line_number":284,"context_line":"        :type column:     string"},{"line_number":285,"context_line":"        :param values:    In case of list columns, the values to be deleted"},{"line_number":286,"context_line":"                          from the list of values"},{"line_number":287,"context_line":"                          In case of dict columns, the keys to delete"},{"line_number":288,"context_line":"                          regardless of their value"},{"line_number":289,"context_line":"        :type value:      varies depending on column"},{"line_number":290,"context_line":"        :param keyvalues: For dict columns, the keys to delete when the key\u0027s"},{"line_number":291,"context_line":"                          value matches the argument value"}],"source_content_type":"text/x-python","patch_set":4,"id":"0037cf1b_5ca674ae","line":288,"range":{"start_line":285,"start_character":0,"end_line":288,"end_character":51},"updated":"2023-04-19 12:25:55.000000000","message":"Doesn\u0027t command already do what db_clear() is trying to do now ?","commit_id":"831ae23f49a044b5bf6ab6f18ce08915be03fd3a"},{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"c7ec741bf0f44a17d33ed19d344ffc881827de65","unresolved":false,"context_lines":[{"line_number":282,"context_line":"        :type record:     string"},{"line_number":283,"context_line":"        :param column:    The column whose value should be deleted"},{"line_number":284,"context_line":"        :type column:     string"},{"line_number":285,"context_line":"        :param values:    In case of list columns, the values to be deleted"},{"line_number":286,"context_line":"                          from the list of values"},{"line_number":287,"context_line":"                          In case of dict columns, the keys to delete"},{"line_number":288,"context_line":"                          regardless of their value"},{"line_number":289,"context_line":"        :type value:      varies depending on column"},{"line_number":290,"context_line":"        :param keyvalues: For dict columns, the keys to delete when the key\u0027s"},{"line_number":291,"context_line":"                          value matches the argument value"}],"source_content_type":"text/x-python","patch_set":4,"id":"826d0a0c_49449fee","line":288,"range":{"start_line":285,"start_character":0,"end_line":288,"end_character":51},"in_reply_to":"0037cf1b_5ca674ae","updated":"2023-04-19 13:13:44.000000000","message":"You are absolutely right. I misread the docs. db_remove should be used instead! Three chears for Lucas!","commit_id":"831ae23f49a044b5bf6ab6f18ce08915be03fd3a"}]}