На code review проверяют корректность и понятность изменений, а у SQL особенно — план, объём работы, блокировки и безопасность миграции. Форматирование лучше автоматизировать.
[имя] спросил: Как вы проводите code review и на что обращаете внимание при ревью SQL-миграций и запросов?
На code review проверяют корректность и понятность изменений, а у SQL особенно — план, объём работы, блокировки и безопасность миграции. Форматирование лучше автоматизировать.
Короткий ответ
Что ответить на собеседовании
Подробный разбор
Ответ с пояснениями
На code review проверяют корректность и понятность изменений, а у SQL особенно — план, объём работы, блокировки и безопасность миграции. Форматирование лучше автоматизировать.
Начните с цели и бизнес-инвариантов, затем проверьте ошибки, конкуренцию и тесты. Для запроса оцените фильтры, индексы, количество возвращаемых строк, пагинацию и параметризацию.
Для миграции нужны размер таблицы, возможные долгие блокировки, совместимость старой и новой версии приложения, поэтапный backfill и план восстановления. Проверять тяжёлые изменения следует на репрезентативных данных, не запускать опасный EXPLAIN ANALYZE для записи прямо на production.