)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34930,"name":"Jianjian Huo","email":"jhuo@nvidia.com","username":"jhuo"},"change_message_id":"ab32c7c262ba76868e47ff313bdda96b11b85243","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5a7ea30f_ad46e46e","updated":"2026-03-17 18:32:32.000000000","message":"thanks for the help!","commit_id":"bcd441cefbb15d572b21e6e82abb4115d71907fd"}],"test/probe/test_reconstructor_rebuild.py":[{"author":{"_account_id":34930,"name":"Jianjian Huo","email":"jhuo@nvidia.com","username":"jhuo"},"change_message_id":"ab32c7c262ba76868e47ff313bdda96b11b85243","unresolved":true,"context_lines":[{"line_number":854,"context_line":"            except direct_client.DirectClientException:"},{"line_number":855,"context_line":"                pass  # some nodes may not have fragments"},{"line_number":856,"context_line":"        # all devs have two distinct frags"},{"line_number":857,"context_line":"        self.assertEqual("},{"line_number":858,"context_line":"            {dev: 2 for dev in dev_frag_etags.keys()},"},{"line_number":859,"context_line":"            {dev: len(set(etags)) for dev, etags in dev_frag_etags.items()},"},{"line_number":860,"context_line":"            dev_frag_etags"}],"source_content_type":"text/x-python","patch_set":1,"id":"43562443_f122696e","line":857,"updated":"2026-03-17 18:32:32.000000000","message":"this is very good improvement. Previous code had two ``direct_get`` in same ``try/except`` block, and we could miss checking one node if it ony has v1 durable fragment.\n\nbut this new change misses one check: ``self.assertEqual(self.frag_etags[node[\u0027index\u0027]], etag)\n``, I added it when I squashed the patch.","commit_id":"bcd441cefbb15d572b21e6e82abb4115d71907fd"}]}