Месяц апрувил код, не понимая половины. Потом попросил
Почти месяц я ставил апрув на код бэкендера, в котором толком ничего не понимал.
Парень пришел с опытом работы с базами под большой нагрузкой, а мы как раз уперлись в то, что база на пиках еле дышит. Он переписывал запросы, добавлял индексы, крутил какие-то настройки, а я смотрел и половины не понимал. Тесты зеленые, графики стали лучше, ну и жал «апрув».
Потом до меня дошло, что ревью у нас по факту нет, есть только моя подпись. И если он когда-нибудь накосячит, я этого даже не замечу.
Написал ему в личку как есть: слушай, я в этой теме плаваю, а проверять тебя все равно нужно. Расскажешь, что ты там вообще делаешь?
Он ответил: «Да без проблем, давай в четверг созвонимся на часик».
В итоге мы так созванивались раз в неделю на протяжении пары месяцев. Он брал свои же пулл-реквесты и показывал: вот тут запрос тормозил, вот почему нужен этот индекс, вот куда смотреть в плане выполнения. Ни разу не дал понять, что я задаю глупые вопросы, хотя задавал я их много.
Писать так, как он, я до сих пор не умею. Но на прошлой неделе впервые нашел у него ошибку на ревью. Он поправил и написал: «О, круто, что заметил».