)]}'
{"swift/common/db.py":[{"author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"change_message_id":"07e9dfe24b4ce2e25f5f99aa00997cb8e76948ae","unresolved":true,"context_lines":[{"line_number":489,"context_line":"            if e.errno \u003d\u003d errno.ENOENT and not os.path.exists(self.db_dir):"},{"line_number":490,"context_line":"                # race to quarantine? w/e, it\u0027s gone"},{"line_number":491,"context_line":"                detail +\u003d \", but it\u0027s already gone\""},{"line_number":492,"context_line":"            else:"},{"line_number":493,"context_line":"                if e.errno not in (errno.EEXIST, errno.ENOTEMPTY):"},{"line_number":494,"context_line":"                    raise"},{"line_number":495,"context_line":"                quar_path \u003d \"%s-%s\" % (quar_path, uuid4().hex)"}],"source_content_type":"text/x-python","patch_set":1,"id":"a549c9de_d8ba9a72","line":492,"updated":"2022-05-02 06:49:40.000000000","message":"If it\u0027s ENOENT but does exist (not sure how that happens) then it wont trigger the raise. also I wonder if rather then if;else -\u003e if; else. We should just elif here?\n\n  if e.errno \u003d\u003d errno.ENOENT and not os.path.exists(self.db_dir):\n    ...\n  elif e.errno not in (errno.EEXIST, errno.ENOTEMPTY):\n    raise\n  ...","commit_id":"a2587ae237e16f118bacea891f9ff1b182692ac5"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"e43e4cdf10134e24e623145821ba400a4fb73d9d","unresolved":true,"context_lines":[{"line_number":489,"context_line":"            if e.errno \u003d\u003d errno.ENOENT and not os.path.exists(self.db_dir):"},{"line_number":490,"context_line":"                # race to quarantine? w/e, it\u0027s gone"},{"line_number":491,"context_line":"                detail +\u003d \", but it\u0027s already gone\""},{"line_number":492,"context_line":"            else:"},{"line_number":493,"context_line":"                if e.errno not in (errno.EEXIST, errno.ENOTEMPTY):"},{"line_number":494,"context_line":"                    raise"},{"line_number":495,"context_line":"                quar_path \u003d \"%s-%s\" % (quar_path, uuid4().hex)"}],"source_content_type":"text/x-python","patch_set":1,"id":"a5639b57_847f2b75","line":492,"in_reply_to":"a549c9de_d8ba9a72","updated":"2022-05-03 17:45:18.000000000","message":"\u003e If it\u0027s ENOENT but does exist (not sure how that happens)\n\nI think you could get that if your quarantine dir disappears between makedirs_count and rename -- though it\u0027d have to happen *twice*.\n\n\u003e then it wont trigger the raise.\n\nEh? Sure it would -- we\u0027ll go into the else, then ENOENT not in (errno.EEXIST, errno.ENOTEMPTY), so raise.\n\nThe elif seems reasonable, though -- I can push that up.","commit_id":"a2587ae237e16f118bacea891f9ff1b182692ac5"},{"author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"change_message_id":"c6bdb0f64ac8ac10c4bc4f9801bb7418c8d31f12","unresolved":true,"context_lines":[{"line_number":489,"context_line":"            if e.errno \u003d\u003d errno.ENOENT and not os.path.exists(self.db_dir):"},{"line_number":490,"context_line":"                # race to quarantine? w/e, it\u0027s gone"},{"line_number":491,"context_line":"                detail +\u003d \", but it\u0027s already gone\""},{"line_number":492,"context_line":"            else:"},{"line_number":493,"context_line":"                if e.errno not in (errno.EEXIST, errno.ENOTEMPTY):"},{"line_number":494,"context_line":"                    raise"},{"line_number":495,"context_line":"                quar_path \u003d \"%s-%s\" % (quar_path, uuid4().hex)"}],"source_content_type":"text/x-python","patch_set":1,"id":"f88b05e6_22a1bd32","line":492,"in_reply_to":"a5639b57_847f2b75","updated":"2022-05-05 05:55:03.000000000","message":"Oh yeah, I guess it would\u0027ve. Somehow I missed that.","commit_id":"a2587ae237e16f118bacea891f9ff1b182692ac5"}]}