log4j/log4cxx + async

Вынеси определение имени в отдельный метод/функцию с кешированием в хэш-таблице. Будет долго резолвиться первый раз. Можно через некоторое время имя инвалидировать

А если DNS сервер упадет? Клиенты будут ждать реквеста пока мой логгер мучается с gethostbyaddr? Ерунда получается..

Посмотрел в коды log4cxx и вижу что важны методы

AppenderSkeleton::doAppend(LoggingEventPtr) которая зовет виртуальную append(LoggingEventPtr), которая в свою очередь в AsyncAppender добавляет LoggingEvent в очередь.

Все это конечно чудесно, но LogginEvent уже в себе имеет String message и методы которые зовутся в AsyncAppender в другом трэде это getMessage/getRenderedMessage, а эти методы нифига не виртуальные в LoggingEvent.

Ерунда получается немного. Легко асинхронично писать в лог котовые String message, но трудно асинхронично формировать эти самые String message.

Пока единственное что приходит в голову это сделать собственый трэд с очередью на gethostbyaddr.

> А если DNS сервер упадет? Клиенты будут ждать реквеста пока мой логгер мучается с gethostbyaddr? Ерунда получается..

Кэш в отдельном треде набивай

> Пока единственное что приходит в голову это сделать собственый трэд с очередью на gethostbyaddr.

Скажи честно, тебе в кишках log4cxx охота копаться? ;)

> Кэш в отдельном треде набивай
Если будет трэд то в принципе неважно есть кэш или нет особо. ИМХО конечно.

> Скажи честно, тебе в кишках log4cxx охота копаться? ;)
А с какого бодуна я тогда его вообще использую если такие вещи приходится делать руками. :-\

> Если будет трэд то в принципе неважно есть кэш или нет особо. ИМХО конечно.

А ты тред на добавление сообщений в лог хочешь или тред для резолвера?

Я бы сделал так: вызываем свою функцию резолвинга для адреса, если в хэше записи нет, то делаем пустую запись (вставляем ключ), в лог пишем только голый ip. В отдельном треде раз в какое-то время проходимся по хэшу, для всех пустых записей резолвим имя. Ну ещё можно инвалидацию делать время от времени для старых записей, если сервер расчитан на работу без перезапусков месяцами. Если имя в логе позарез нужно, то резолвить сразу.

> А с какого бодуна я тогда его вообще использую если такие вещи приходится делать руками. :-\

Ну концепция у log4j ничё так, вполне интересная.

> А ты тред на добавление сообщений в лог хочешь или тред для резолвера?
Для резолвера - я пишу в файл и это относительно быстро.

Ладно напишу собственный трэд который читает очередь записей для лога и формирует String message, резолвя (во слово-то..) адрес.

Жалко что Layout'ы накладываются перед AsyncAppender::append().. Если бы накладывались позднее - достаточно было бы написать свой Layout с резолвингом.