)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"2e30bd3f46757468e8533b26830c0964fa243159","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In current implementation, immediately after a reconciler deleted empty"},{"line_number":10,"context_line":"reconciler-container, container-replicators enqueue rows to brokers"},{"line_number":11,"context_line":"which are considered to be deleted."},{"line_number":12,"context_line":"This patch fixes container-replicators to update brokers\u0027 put-timestamp."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I37856b9bf0881cc82d4b242c163cedc8ecbfe419"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3afc51ec_357662a4","line":11,"updated":"2016-03-15 07:16:03.000000000","message":"When container-replicators tried to enqueue rows to reconciler db, if the db file exists already and it is in is_deleted status, recoinciler broker will add the rows into the deleted db. I got the message from Kazuhiro via F2F conversation.","commit_id":"5c10d570ef64a507786736a47f8ac0c1f234aae3"}],"swift/container/replicator.py":[{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"2e30bd3f46757468e8533b26830c0964fa243159","unresolved":false,"context_lines":[{"line_number":124,"context_line":"                broker.initialize(timestamp, 0)"},{"line_number":125,"context_line":"            except DatabaseAlreadyExists:"},{"line_number":126,"context_line":"                pass"},{"line_number":127,"context_line":"        broker.update_put_timestamp(timestamp)"},{"line_number":128,"context_line":"        if self.reconciler_containers is not None:"},{"line_number":129,"context_line":"            self.reconciler_containers[container] \u003d part, broker, node[\u0027id\u0027]"},{"line_number":130,"context_line":"        return broker"}],"source_content_type":"text/x-python","patch_set":2,"id":"3afc51ec_723920f2","line":127,"updated":"2016-03-15 07:16:03.000000000","message":"broker.initialize takes a arg, \u0027put_timestamp\u0027, at 3 lines above and then it will be set as put_timestamp in container_info table so this added line seems meaningless because update_put_timestamp updates the put_timestamp column at the container_info table...\n\n\n\nAh, ok you means like:\n\n if not os.path.exists(broker.db_file):\n   \u003csnip\u003e\n elif broker.is_deleted():\n   broker.update_put_timestamp(timestamp)","commit_id":"5c10d570ef64a507786736a47f8ac0c1f234aae3"}]}