fix: speed up academic degree filter #32
Reference in New Issue
Block a user
Delete Branch "fix/academic-degree-filter-performance"
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?
Фильтр переведён с поиска по JSONB на индексируемый признак has_academic_degree. Добавлены миграция и runtime-backfill существующих данных.
Независимое ревью: есть blocking finding.
migrations/009_academic_degree_filter.sql: шаблонph\\.?[[:space:]]*d\\.?в PostgreSQL получает двойной обратный слеш. В POSIX-регэкспе это ищет буквальный\, а не экранированную точку, поэтому существующие записи сPhD/Ph.D.при миграции останутсяhas_academic_degree = FALSEи будут ошибочно исключаться до следующего краулинга. Нужен тот же шаблон с одиночным\.(как фактически передаёт runtime-backfill) и тест миграционного/backfill-пути на PhD.Проверены diff, runtime-backfill, индексы и тесты.
compileallпройден; pytest в окружении отсутствует.Повторное независимое ревью актуального PR выполнено. Предыдущее blocking-замечание устранено: в
migrations/009_academic_degree_filter.sqlиспользуются одиночные\.; шаблон корректно сопоставляетPhDиPh.D., как и runtime-backfill. Полный diff повторно проверен: новых блокирующих проблем в корректности, runtime-backfill, миграции, индексе, производительности или безопасности не найдено.compileallпройден; pytest в окружении отсутствует.Финальное независимое read-only ревью завершено.
CHANGELOG.mdсодержит корректный раздел0.7.4с пользовательским описанием ускорения фильтра. Полный актуальный diff повторно проверен, включая исправление SQL-регэкспа для PhD/Ph.D.; blocking findings отсутствуют.