Skip to content

fix: disable logging from non-main threads because of performance problems

Georgy Moshkin requested to merge gmoshkin/disable-thread-safe-logger into master

Summary

  • fix: disable logging from non-main threads because of performance problems

Проблема в том, что тред-сейф логгер в отдельном файбере поллит cbus канал, в которых приходят сообщения из других тредов https://git.picodata.io/picodata/picodata/picodata/-/blob/9d52b5820d8f450cbc5ce1e09157fc370d0d1add/src/tlog.rs#L225. Это приводит к регулярному вызову fiber_cond_wait_timeout: https://git.picodata.io/picodata/picodata/tarantool-module/-/blob/407270bff2eb966c8c9af732e1af32f28b06cbc3/tarantool/src/cbus/unbounded.rs#L64

Как показывает флеймграф с запусков бенчмарков https://git.picodata.io/picodata/plugin/redisproto/-/merge_requests/15 fiber_cond_wait_timeout приводящий к таймауту реализован крайне не эффективно image

Частых вызовов fiber_cond_wait_timeout нужно избегать.

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

  • Cherry-pick to: none
  • Docs follow-up: not necessary
Edited by Georgy Moshkin

Merge request reports