)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"d05f79ae9a6568ac641c0f2fc77104e42f4a0311","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Clay Gerrard \u003cclay.gerrard@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-02-09 14:36:01 -0600"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Shink the public shrinking interface"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We consume find_compactable_shard_sequences to produce actionable shard"},{"line_number":10,"context_line":"range sequences, and return them to the caller to report and confirm."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7171f4e4_02a82991","line":7,"updated":"2021-02-10 00:40:34.000000000","message":"nit: shrink\n\nAlso, I\u0027m not sure this is really shrinking the interface much... we\u0027ve still got the two component functions... we\u0027ve just added a new one.","commit_id":"210cb17393b124fbce2d46d95185ff1884b13655"}],"swift/container/sharder.py":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"d05f79ae9a6568ac641c0f2fc77104e42f4a0311","unresolved":true,"context_lines":[{"line_number":317,"context_line":""},{"line_number":318,"context_line":""},{"line_number":319,"context_line":"def process_and_finalize_compactable_shard_sequences("},{"line_number":320,"context_line":"        broker, compactable_sequences, timestamp\u003dNone):"},{"line_number":321,"context_line":"    timestamp \u003d timestamp or Timestamp.now()"},{"line_number":322,"context_line":"    acceptor_ranges, shrinking_ranges \u003d process_compactable_shard_sequences("},{"line_number":323,"context_line":"        compactable_sequences, timestamp)"}],"source_content_type":"text/x-python","patch_set":1,"id":"93bad868_44270eaf","line":320,"range":{"start_line":320,"start_character":39,"end_line":320,"end_character":53},"updated":"2021-02-10 00:40:34.000000000","message":"Nobody actually uses this, though, yeah? YAGNI? Er, AWGNI?","commit_id":"210cb17393b124fbce2d46d95185ff1884b13655"},{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"ea6e3963ccd1fa96833c38a9553cf785c783f1c6","unresolved":true,"context_lines":[{"line_number":317,"context_line":""},{"line_number":318,"context_line":""},{"line_number":319,"context_line":"def process_and_finalize_compactable_shard_sequences("},{"line_number":320,"context_line":"        broker, compactable_sequences, timestamp\u003dNone):"},{"line_number":321,"context_line":"    timestamp \u003d timestamp or Timestamp.now()"},{"line_number":322,"context_line":"    acceptor_ranges, shrinking_ranges \u003d process_compactable_shard_sequences("},{"line_number":323,"context_line":"        compactable_sequences, timestamp)"}],"source_content_type":"text/x-python","patch_set":1,"id":"064a31f9_a0e39d88","line":320,"range":{"start_line":320,"start_character":39,"end_line":320,"end_character":53},"in_reply_to":"93bad868_44270eaf","updated":"2021-02-10 14:36:54.000000000","message":"i can\u0027t figure out AWGNI","commit_id":"210cb17393b124fbce2d46d95185ff1884b13655"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"2adeb67da8934dd4d7970d3547d0e3eed1f76267","unresolved":true,"context_lines":[{"line_number":321,"context_line":"    timestamp \u003d timestamp or Timestamp.now()"},{"line_number":322,"context_line":"    acceptor_ranges, shrinking_ranges \u003d process_compactable_shard_sequences("},{"line_number":323,"context_line":"        compactable_sequences, timestamp)"},{"line_number":324,"context_line":"    finalize_shrinking(broker, acceptor_ranges, shrinking_ranges, timestamp)"},{"line_number":325,"context_line":""},{"line_number":326,"context_line":""},{"line_number":327,"context_line":"class CleavingContext(object):"}],"source_content_type":"text/x-python","patch_set":1,"id":"4622a727_0ea7edae","line":324,"updated":"2021-02-10 10:03:03.000000000","message":"in case you also wonder why finalize_shrinking exists, it is used by repair:\n\nhttps://review.opendev.org/c/openstack/swift/+/765624/15/swift/cli/manage_shard_ranges.py#638","commit_id":"210cb17393b124fbce2d46d95185ff1884b13655"},{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"ea6e3963ccd1fa96833c38a9553cf785c783f1c6","unresolved":true,"context_lines":[{"line_number":321,"context_line":"    timestamp \u003d timestamp or Timestamp.now()"},{"line_number":322,"context_line":"    acceptor_ranges, shrinking_ranges \u003d process_compactable_shard_sequences("},{"line_number":323,"context_line":"        compactable_sequences, timestamp)"},{"line_number":324,"context_line":"    finalize_shrinking(broker, acceptor_ranges, shrinking_ranges, timestamp)"},{"line_number":325,"context_line":""},{"line_number":326,"context_line":""},{"line_number":327,"context_line":"class CleavingContext(object):"}],"source_content_type":"text/x-python","patch_set":1,"id":"65056889_b81ed3bd","line":324,"in_reply_to":"4622a727_0ea7edae","updated":"2021-02-10 14:36:54.000000000","message":"i really had assumed as much; thanks for the pointer","commit_id":"210cb17393b124fbce2d46d95185ff1884b13655"}]}