docs(tradein): снять устаревшее обоснование обрыва прогона Домклика (#2854) #2859

Merged

bot-backend merged 1 commit from docs/2854-stale-justifications into main

2026-08-13 07:34:00 +00:00

bot-backend commented

2026-08-13 07:28:32 +00:00

Collaborator

Обоснование пережило свою предпосылку

В докстринге domclick_detail_backfill.py стояло:

No IP-rotation/cooldown recovery step exists here (DomClick uses one dedicated residential proxy, not a rotating pool)

Вторая половина перестала быть правдой на миграции 253 (#2800), которая явно сняла резервацию узла. На 13.08 узлов четыре, у всех provider_affinity='any' — проверено на проде.

Разница не косметическая. Пока узел был один, «ротации нет» следовало из «ротировать нечего», и обрыв прогона на первом блоке был вынужденным. Теперь это непринятое решение — а обоснование, пережившее свою предпосылку, читается как довод не менять поведение. Ровно так я сам вчера прочитал его как объяснение и едва не закрыл вопрос.

Цена измерена в #2854: buckets_completed = 0 во всех прогонах свипа, то есть шесть комнатных корзин не доводятся никогда, а блок бьёт внутри первой. Длительность до блока 111-332 секунды при примерно постоянной выработке — свежий узел, судя по этому, получает свой бюджет.

Поведение не меняется: правка снимает ложный довод, а не принимает решение за владельца.

Заодно — мой собственный след

Шапка 260_houses_drop_has_panorama.sql называла себя 259_. Это вчерашнее переименование при коллизии номеров: два независимых PR взяли 259, я поправил имя файла, ссылку в тесте и запись в манифесте, а комментарий внутри пропустил.

Правка применённой миграции безопасна: _schema_migrations — это (filename TEXT PRIMARY KEY, applied_at), без контрольной суммы содержимого, проверено в deploy-tradein.yml:461. Файл повторно не применится.

Проверка

Только комментарии и докстринг — поведение не затронуто, тестов не требуется. ruff чист.

Refs #2854, #2800, #2857

## Обоснование пережило свою предпосылку В докстринге `domclick_detail_backfill.py` стояло: > No IP-rotation/cooldown recovery step exists here **(DomClick uses one dedicated residential proxy, not a rotating pool)** Вторая половина перестала быть правдой на миграции **253** (#2800), которая явно сняла резервацию узла. На 13.08 узлов четыре, у всех `provider_affinity='any'` — проверено на проде. **Разница не косметическая.** Пока узел был один, «ротации нет» *следовало* из «ротировать нечего», и обрыв прогона на первом блоке был вынужденным. Теперь это непринятое решение — а обоснование, пережившее свою предпосылку, читается как довод не менять поведение. Ровно так я сам вчера прочитал его как объяснение и едва не закрыл вопрос. Цена измерена в #2854: `buckets_completed = 0` во **всех** прогонах свипа, то есть шесть комнатных корзин не доводятся никогда, а блок бьёт внутри первой. Длительность до блока 111-332 секунды при примерно постоянной выработке — свежий узел, судя по этому, получает свой бюджет. Поведение **не меняется**: правка снимает ложный довод, а не принимает решение за владельца. ## Заодно — мой собственный след Шапка `260_houses_drop_has_panorama.sql` называла себя `259_`. Это вчерашнее переименование при коллизии номеров: два независимых PR взяли 259, я поправил имя файла, ссылку в тесте и запись в манифесте, а комментарий внутри пропустил. Правка применённой миграции безопасна: `_schema_migrations` — это `(filename TEXT PRIMARY KEY, applied_at)`, без контрольной суммы содержимого, проверено в `deploy-tradein.yml:461`. Файл повторно не применится. ## Проверка Только комментарии и докстринг — поведение не затронуто, тестов не требуется. `ruff` чист. Refs #2854, #2800, #2857

bot-backend added 1 commit 2026-08-13 07:28:32 +00:00

docs(tradein): снять устаревшее обоснование обрыва прогона Домклика

CI / changes (pull_request) Successful in 9s

Details

CI Trade-In / changes (pull_request) Successful in 8s

Details

CI Trade-In / browser-tests (pull_request) Has been skipped

Details

CI Trade-In / frontend-checks (pull_request) Has been skipped

Details

CI / backend-tests (pull_request) Has been skipped

Details

CI / frontend-tests (pull_request) Has been skipped

Details

CI / openapi-codegen-check (pull_request) Has been skipped

Details

CI Trade-In / backend-tests (pull_request) Successful in 4m18s

Details

062eee9b16

Докстринг domclick_detail_backfill утверждал: «No IP-rotation/cooldown recovery
step exists here (DomClick uses one dedicated residential proxy, not a rotating
pool)». Второе перестало быть правдой на миграции 253 (#2800), снявшей
резервацию узла: на 13.08 узлов четыре и у всех provider_affinity='any'.

Разница не косметическая. Пока узел был один, «ротации нет» СЛЕДОВАЛО из
«ротировать нечего» — обрыв прогона на первом блоке был вынужденным. Теперь
это непринятое решение, и обоснование, пережившее свою предпосылку, читается
как довод не менять поведение.

Цена измерена в #2854: buckets_completed=0 во ВСЕХ прогонах свипа, то есть
шесть комнатных корзин не доводятся никогда, а блок бьёт внутри первой.
Длительность до блока 111-332 с при примерно постоянной выработке — свежий
узел, вероятно, получает свой бюджет.

Заодно: шапка 260_houses_drop_has_panorama.sql называла себя 259_ — след
вчерашнего переименования при коллизии номеров (два PR взяли 259; имя файла,
ссылку в тесте и манифест поправил, комментарий внутри пропустил). Отслеживание
идёт по filename (_schema_migrations, без контрольной суммы), правка безопасна.

Refs #2854, #2800