)]}'
{"test/probe/test_sharder.py":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"1fb800e5dc231bb478472332178d828cd796af22","unresolved":true,"context_lines":[{"line_number":3379,"context_line":"            \u0027dir_content\u0027: dir_content,"},{"line_number":3380,"context_line":"        }"},{"line_number":3381,"context_line":""},{"line_number":3382,"context_line":"    def test_handoff_does_not_reset_own_shard_rance(self):"},{"line_number":3383,"context_line":"        ctx \u003d self._setup_shared_container()"},{"line_number":3384,"context_line":""},{"line_number":3385,"context_line":"        # let\u0027s replicate"}],"source_content_type":"text/x-python","patch_set":1,"id":"faeed1dd_2d8511ca","line":3382,"updated":"2021-09-17 18:23:07.000000000","message":"I think this passes on master","commit_id":"c1737b3423ded922e44e29946b2e0ea1e6c16855"},{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"1fb800e5dc231bb478472332178d828cd796af22","unresolved":true,"context_lines":[{"line_number":3392,"context_line":"        handoff_osr \u003d handoff_broker.get_own_shard_range(no_default\u003dTrue)"},{"line_number":3393,"context_line":"        self.assertIsNotNone(handoff_osr.epoch)"},{"line_number":3394,"context_line":""},{"line_number":3395,"context_line":"    def test_rebalance_does_not_reset_own_shard_rance(self):"},{"line_number":3396,"context_line":"        ctx \u003d self._setup_shared_container()"},{"line_number":3397,"context_line":""},{"line_number":3398,"context_line":"        # OK so handoff don\u0027t cause resets and we can\u0027t load it up with"}],"source_content_type":"text/x-python","patch_set":1,"id":"a7c400d7_d0a167a4","line":3395,"updated":"2021-09-17 18:23:07.000000000","message":"this one too","commit_id":"c1737b3423ded922e44e29946b2e0ea1e6c16855"},{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"1fb800e5dc231bb478472332178d828cd796af22","unresolved":true,"context_lines":[{"line_number":3459,"context_line":"        # osr\u0027s but we are now blocking merging epoch with no epoch. Why,"},{"line_number":3460,"context_line":"        # because the lifecycle of a sharded container once it gets an epoch"},{"line_number":3461,"context_line":"        # it should never be lost."},{"line_number":3462,"context_line":"        self.assertIsNotNone(old_osr.epoch)"},{"line_number":3463,"context_line":"        self.assertEqual(old_osr.state, ShardRange.SHARDED)"},{"line_number":3464,"context_line":""},{"line_number":3465,"context_line":"        # run the sharder so the primary shards and grows the epoch in the"}],"source_content_type":"text/x-python","patch_set":1,"id":"c48a3f49_37ee21aa","line":3462,"updated":"2021-09-17 18:23:07.000000000","message":"I\u0027d guess on master this should fail because replicaiton of reset_osr.epoch???  Is this what the parent is trying to fix?","commit_id":"c1737b3423ded922e44e29946b2e0ea1e6c16855"}],"test/unit/container/test_sharder.py":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"1fb800e5dc231bb478472332178d828cd796af22","unresolved":true,"context_lines":[{"line_number":4800,"context_line":"                         ShardRange.SHRUNK):"},{"line_number":4801,"context_line":"                epoch \u003d None"},{"line_number":4802,"context_line":"            else:"},{"line_number":4803,"context_line":"                epoch \u003d Timestamp.now()"},{"line_number":4804,"context_line":""},{"line_number":4805,"context_line":"            # each state test starts with a fresh no-own-sr-having broker"},{"line_number":4806,"context_line":"            broker \u003d self._make_broker(container\u003d\u0027c-%s\u0027 % state)"}],"source_content_type":"text/x-python","patch_set":1,"id":"a2ff2644_c1d9ead4","line":4803,"updated":"2021-09-17 18:23:07.000000000","message":"why do we let osr w/o epoch into the shard range table at all?  what does the fact that it\u0027s a brand new broker have to do with that state being in anyway sane?","commit_id":"c1737b3423ded922e44e29946b2e0ea1e6c16855"}]}