)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"9a646ab2920a4ec6b54f0d56098a06c26151b224","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"464f781c_7efacb66","updated":"2023-04-21 18:22:33.000000000","message":"this looks kind ok, once we have the db_state hint it\u0027s not so bad to think we might hit memcache instead of the root for every update.","commit_id":"6862a5a9389c16a6f4b0c1ab4b24ba780be7dac7"}],"swift/obj/updater.py":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"9a646ab2920a4ec6b54f0d56098a06c26151b224","unresolved":true,"context_lines":[{"line_number":617,"context_line":"        container keys are always the root."},{"line_number":618,"context_line":"        \"\"\""},{"line_number":619,"context_line":"        acct, cont \u003d split_update_path(update)"},{"line_number":620,"context_line":"        if self.memcache and update.get(\u0027db_state\u0027) in (SHARDED, SHARDING) \\"},{"line_number":621,"context_line":"                and not update.get(\u0027container_path\u0027):"},{"line_number":622,"context_line":"            # We are targeting a root container directly, let\u0027s try and bypass"},{"line_number":623,"context_line":"            # it by attempting to pull update shard list from memcache"}],"source_content_type":"text/x-python","patch_set":5,"id":"905c34d6_601f829c","line":620,"updated":"2023-04-21 18:22:33.000000000","message":"less obvious in this context that \"db_state\" \u003d\u003d \"sharding_state\"\n\nI guess I was just thinking that in the proxy we hve code that maintains this list of states and now here in the updater as well\n\nI think if we just say `is_sharded: boolean` it\u0027s a little closer to SSOT","commit_id":"6862a5a9389c16a6f4b0c1ab4b24ba780be7dac7"}]}