@RGJ 저는 그 부분을 전혀 변경하지 않았습니다. Upload를 다시 생성한 것은 @zogstrip이 https://meta.discourse.org/t/cannot-execute-the-rake-uploads-migrate-from-s3/78541/11에서 수행한 버그 수정이었습니다.
저는 이미 존재하던 코드가 동작하도록 하려는 것뿐이고, discourse 내부 구조를 잘 알지 못합니다. 어둠 속에서 헤매는 일이 많았습니다. 제 Ruby 경험은 discourse를 위해 작업한 몇 가지 PR 정도입니다. 기존 코드의 패턴을 따르는 것이 가장 효율적인 방법이 아닌 것 같습니다(단축 평가에 대해 @pfaffman과 나눈 위의 대화를 참고하세요). 전적으로 동의합니다. 오늘早些까지 OptimizedImages.url도 /uploads 경로로 수정되고 etag가 nil로 설정되어야 한다는 사실조차 깨닫지 못했다는 점으로 보아, 저는 여전히 눈감고 운전하는 상태입니다.
최소限으로 게시글 내의 기존 리터럴 URL을 수정하기 위해 게시글을 순회하는 작업이 여전히 필요합니다. 게시글을 재검증하지 않고, 오류를 조용히 무시하지 않는 등 다른 수정 사항들도 여전히 필요합니다. 또한 라이브 사이트への 영향을 제한하기 위해 레이트 리미팅을 적용하는 것도 여전히 바람직하다고 생각합니다.
게시글이 아닌 업로드 업데이트와 관련된 두 가지 우려 사항에 대해, 제 진행 중인 작업(아직 라이브 사이트에서 테스트되지 않음) (커밋)이 도움이 될 수 있을 것입니다. 다만 게시글 업로드 마이그레이션이 모두 완료될 때까지 제 라이브 사이트에서 테스트할 수 없습니다.
예전에 잘 작동하던 것을 조정하는 것 정도만 감당할 수 있었습니다. 더 빠른 마이그레이션을 원하신다면 전적으로 찬성합니다. 제 느리지만 적어도 더 나은 작업을 파레토 개선(Pareto improvement)으로 병합한 후, 그것을 훨씬 더 나은 것으로 완전히 교체하는 것이 합리적일 수 있으며, 그때 저는 그것을 사용할 위치에 있지 않더라도 가장 먼저 축하할 것입니다.