Migrate_from_s3 problems

@RGJ Я вообще ничего не менял; переделка Upload — это исправление ошибки от @zogstrip, описанное здесь: Cannot execute the rake uploads:migrate_from_s3 - #11

Я просто пытаюсь заставить работать код, который уже был, но плохо знаком с внутренним устройством Discourse; я долго действовал вслепую. Мой опыт работы с Ruby ограничивается несколькими моими PR для Discourse. Следование паттернам существующего кода, как мне кажется, не самый эффективный путь (см. мою переписку с @pfaffman выше о коротком замыкании), и я полностью согласен. Как видно из того, что даже сегодня утром я не сразу понял, что OptimizedImages.url тоже нужно изменить на путь /uploads, а etag установить в nil (и, возможно, ещё что-то), я всё ещё действовал вслепую.

Пока что всё равно нужно сначала пройти по всем постам, по крайней мере для исправления старых буквенных URL в постах. Также нужны некоторые другие исправления, например, чтобы не переутверждать посты и не молча игнорировать ошибки. Думаю, всё же стоит внедрить ограничение скорости запросов, чтобы минимизировать влияние на работающие сайты.

По вашим первым двум замечаниям, касающимся обновлений, не связанных с постами, вот моя работа в процессе, ещё не протестированная на рабочем сайте (коммит), которая, возможно, поможет. Однако я приступлю к тестированию на своём рабочем сайте только после завершения миграции загрузок в постах.

Я хотел лишь немного подправить то, что раньше работало. Если вы хотите реализовать более быструю миграцию — я только «за». Возможно, имеет смысл сначала объединить мою более медленную, но всё же улучшенную версию как парето-улучшение, а затем полностью заменить её на что-то гораздо лучшее. Я буду первым, кто порадуется этому, даже если к тому моменту я уже не смогу использовать это решение.

1 лайк