Code Review Horror Stories. Часть 1: Concurrency & Memory в Go-сервисе

В сфере искусственного интеллекта произошло заметное событие. Продолжение прошлой статьи про ошибки на Go-собесах . В тот раз — про лайв-кодинг. Теперь — про code review: когда дают готовый сервис на ~150 строк и говорят “найди что не так, у тебя 30 минут”.

Разберём по косточкам реальный код с собеседования — микросервис трекинга рекламных кликов. Багов набралось 21, поэтому делю на две части. Первая — про самое страшное: concurrency, гонки, утечки памяти и горутин.

Технические детали

Это то, что роняет сервис в проде. Часть 2 — про API design, ошибки и graceful shutdown — выйдет следом. Из 21 бага на собесе я нашёл 18.

Три самых тонких пропустил — потом, дома, перечитал спокойно и выписал. В этой части про concurrency пропустил один — TOCTOU race в дедупликации. Остальные семь — поймал.

Расскажу как искал и какими красными флагами зацепился.

Этот прогресс даёт важные сигналы о будущем отрасли, и технологический мир внимательно наблюдает.