НЕмитап Golang#1 Алексей Мичурин - Логирование как в Авито: go + slog

Рет қаралды 6,493

AvitoTech

Күн бұрын

Пікірлер: 24

@timur-yusipov 6 ай бұрын

Интересно, спасибо)

@ffff00-korj 3 ай бұрын

Спасибо, интересно как все последовательно приходят к подходу, когда ошибки нужно врапать)

@MichurinTube 3 ай бұрын

Согласен. Но, кажется, к врапанью надо было как-то специально идти лет 7 назад. Сейчас, когда есть %w и вкусняшки типа As, Is и даже errors.Join, - сам язык и стандартная библиотека тебя толкает к врапанью.

@satisataka3437 6 ай бұрын

Спасибо, очень полезно!

@wat4mon 6 ай бұрын

крутой доклад, респект, больше такого контента

@AvitoTech 5 ай бұрын

@qoonmax Ай бұрын

Отличный материал

@victorm7551 6 ай бұрын

спасибо, было интересно

@zenderfufikoff 6 ай бұрын

думаю userId это всё же второстепенный аргумент. Первостепенным должен быть traceId. А так живенько, полезно. Спасибо.

@MichurinTube 5 ай бұрын

TraceId точно надо добавлять, если он есть. Но главное, иметь в логах информацию, которую сообщают пользователи/тестировщики.

@daniil-pershin 6 ай бұрын

Видео огонь, спасибо Алексей, поделитесь, пожалуйста, своим конфигом для vim 🙏

@brothers_karamazovs 5 ай бұрын

Отличный доклад, спасибо!

@Денис-в1я6к 5 ай бұрын

Прикольно смотреть, как меняется подходы к разработке. Ранее вот эти примеры использовались, чтоб на собеседовании спрашивать какие SOLID принципы здесь нарушаются. Очень много вопросов возникает после просмотра, жаль не удалось посмотреть в прямом эфире. - Почему логгер(интерфейсный слой) знает о какой-то бизнес логике? Если проблема лишь в том что какой-то программист захочет назвать поле по другому, что мешает этому программисту создать другую структуру данных? П.С. Из ваших слов в авито вы прокидываете мапу, тогда не понятно зачем пример с структурами... - Зачем засорять контекст данными? Я думаю вытекающие проблемы понятны - Почему выбрали именно slog? Честно ожидал что будет как "Jaeger для трассировки в микросервисной архитектуре", а вышла на аудиторию, которая только начинает работать с golang

@nikolays2490 5 ай бұрын

Леша мой герой!)

@Александр-б6с3ъ 5 ай бұрын

Зачем 15 минут заставок?

@fresonn 7 күн бұрын

Из 7-го глупого примера! В самом верху логировать ошибку, чтобы не забыть залогировать ошибку, вообще-то ошибки еще нужно обрабатывать и в какую архитектуру это влезет(имею в виду все эти 100500 враперов)?

@hopeuesimsure6766 6 ай бұрын

cfbr

@AntonZelinsky 5 ай бұрын

Не возникает делима где логировать 27:05 ошибку, она всегда должна логироваться, она должна логироваться там где происходит. Это бред какой-то логировать её где-то на нижних уровнях. Представь, что ты используешь стороннюю библиотеку и она вместо того чтобы залогировать какую-то ошибку в себе, прокалывание её наверх, чтобы ты сам решал логировать её или нет. Такого я ни разу не видел в жизни. Да и логируется всегда весь контекс и стектрейс, который есть только в месте возникновения ошибки. Я посмотрел лекцию до конца и видел вариант с добавлением в специально созданную структуру контекста, но это лютая дичь! Вернёмся к моему другому примеру с 5000 логов, тебе нужно будет сделать 5000 обвёрток и чуть меньше структур для прокидывания контекста наверх? А сколько принципов программирования и проектирования это нарушает? +-10 штук. А если у тебя часть кода будет лежать в одном модуле, а часть в другом, нужно будет в вызывающем модуле всё логировать? А если его начнут использовать ещё кто-то? Да и бред про устоявшуюся команду меня паразил. А что если ваш сторожила уволится? Да и возвращать ошибку на каждом уровне это ещё тот спагетти. Уже давно выработан подход к обработке ошибок, почитайте книгу Чистый код, главу про обработку ошибок, когда возникает ошибка, она сразу логируется и возвращается исключение, которое перехватывается где-то на верхнем уровне. Да, в го нет исключений, но есть паники, которыми можно реализовать тот же самый подход. Такой бы код (даже с учётом примеров) у меня никогда бы не прошёл ревью. В общем, лекция хорошая для новичков, кто вообще не знает о структурировании легировании, но для опытных она будет бесполезной.

@sergeykosovsky7005 5 ай бұрын

паниками реализовывать исключение - ты чего)) это очень дорого, поэтому бессмысленно

@rostislavteryaev3894 2 ай бұрын

жесть. Люди которые исключения в го делают с использованием паник кого-то ревьюят)

@oxidant5 2 ай бұрын

Представь, что ты используешь стороннюю библиотеку и она вместо того чтобы прокинуть ошибку наверх, чтобы ты сам решал логировать её или нет, логирует её в себе. Такого я ни разу не видел в жизни.

@fresonn 7 күн бұрын

Написано хорошо, со всем согласен, кроме паник.

@AntonZelinsky 5 ай бұрын

Смешно говорит, что код безупречен, а в нём десяток функций которые просто логируют. Так, если в приложении будет 10000 файлов и 5000 комментариев, нужно будет сделать 5000 функций обвёрток для лога?

@MichurinTube 5 ай бұрын

Привет, не совсем так. Одна функция на ключ, а не на файл )