fix(tradein/observability): результат detail-бэкфиллов доезжает до total_seen (#3044) #3096
No reviewers
Labels
No labels
Fable 5 ревью
GG-форсайт
admin
analytics
auth
automation
bug
business
chore
ci
compliance
data
data-moat
docs
duplicate
dx
enhancement
feedback/max
generative
needs-discussion
needs-human
observability
pause-bots
performance
priority/p0
priority/p1
priority/p2
priority/p3
scope/backend
scope/db
scope/devops
scope/frontend
scope/qa
scrapers
security
site-finder
stage/1
stage/2
status/blocked
status/done
status/needs-analysis
status/needs-fix
status/qa
status/ready
status/review
status/wip
tech-debt
tradein
ux
week ревью 1
wontfix
ИРД
вторичка
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: lekss361/gendesign#3096
Loading…
Add table
Reference in a new issue
No description provided.
Delete branch "fix/3044-enriched-total-seen"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Что
Замер в #3044 прочитал «обогащено 0 за 7 дней» у
avito_detail_backfillпри реальных 801 обогащении (listings.detail_enriched_at). Корень: detail-бэкфиллы (avito/yandex/domclick) пишут результат только ключомenrichedв counters, которого_column_countsне знал — колонкаtotal_seenоставалась 0 у всех прогонов, done и banned одинаково. Провал и успех писали один и тот же признак.Как
'enriched'-фолбэк в_column_counts(обе копии: app + scraper-kit) — только для витринной колонки. В_RESULT_COUNTER_KEYSключ намеренно НЕ добавлен: тот список кормит zero-result-стрик, где догнавший очередь бэкфилл (attempted=0, enriched=0) стал бы непрерываемым измеренным нулём — ровно та ловушка, за которую ревью #2703 выкинуло из списка голыйrows_inserted. Первая версия правки добавляла ключ в общий список — её завернули тесты #2703 (2 красных), раскрой пересмотрен.Проверки
test_3044_enriched_result_key.py: красный на origin/main по ЗНАЧЕНИЮ ((None, None)вместо(61, None)), зелёный с фиксом; инвариант-тест закрепляет контракт стрик-сторожа (зелёный в обеих эрах)test_2703_*,test_honest_run_status_*,test_2670_*,test_scrape_skip_visibility)Приёмка после деплоя:
SELECT sum(total_seen) FROM scrape_runs WHERE source='avito_detail_backfill' AND started_at > <деплой>должен стать >0 при первом же прогоне с enriched>0.Closes #3044? Нет — сам дефект (439/QRATOR) остаётся, это только наблюдаемость из последнего комментария.
🤖 Generated with Claude Code