)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":38496,"name":"Andressa Cabistani","display_name":"Andressa","email":"acabistani@gmail.com","username":"andressadotpy","status":"I\u0027m a Software Engineer at Red Hat and I love Open Source and connect with people! Feel free to DM through IRC, I\u0027ll be delighted to chat"},"change_message_id":"f16d5e6c06d7581b8cb48045c383df6f8a587838","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a67d1509_8e1e31eb","updated":"2026-07-28 14:42:51.000000000","message":"Looks good to me! A summary of what I analyzed:\n\n- Great test coverage\n - Tested _history_changed dirty bit: set by first rebalance, reset between passes, set by aging-only pass\n- Tested wall-clock \u003c\u003d boundary: age\u003d1 with max_history_cycles\u003d1 still blocks (the key fix)\n- Tested pass-based \u003c boundary: strict \u003c preserved for min_part_hours\u003d0\n- Tested auto-promote: inert history stays v1, live history promotes to v2, history section present\n- Tested 254 cap: 254 accepted, 255 rejected","commit_id":"a58ae65bd7b4010af7eeedc1ac479ea1a4ca048a"}],"swift/common/ring/builder.py":[{"author":{"_account_id":38496,"name":"Andressa Cabistani","display_name":"Andressa","email":"acabistani@gmail.com","username":"andressadotpy","status":"I\u0027m a Software Engineer at Red Hat and I love Open Source and connect with people! Feel free to DM through IRC, I\u0027ll be delighted to chat"},"change_message_id":"f16d5e6c06d7581b8cb48045c383df6f8a587838","unresolved":true,"context_lines":[{"line_number":227,"context_line":"        byte, bit \u003d divmod(part, 8)"},{"line_number":228,"context_line":"        return bool(self._part_moved_bitmap[byte] \u0026 (128 \u003e\u003e bit))"},{"line_number":229,"context_line":""},{"line_number":230,"context_line":"    def _history_age_is_recent(self, age, threshold):"},{"line_number":231,"context_line":"        # With a wall-clock cycle, a newly-reset cell may inherit almost a full"},{"line_number":232,"context_line":"        # cycle of the shared epoch\u0027s remainder. Keep the threshold boundary"},{"line_number":233,"context_line":"        # itself live so history is never released before a complete retention"}],"source_content_type":"text/x-python","patch_set":3,"id":"2f4a70fb_dd034958","line":230,"updated":"2026-07-28 14:42:51.000000000","message":"Suggestion: rename the method to something like `_history_age_within_retention()`","commit_id":"a58ae65bd7b4010af7eeedc1ac479ea1a4ca048a"}]}