[Tarantool-patches] [PATCH v3 0/3] log: allow json formatter in boottime logger
Oleg Babin
olegrok at tarantool.org
Thu Jul 2 13:08:00 MSK 2020
Hi! LGTM for whole patchset except one note that I've sent separately
for the second patch.
On 02/07/2020 12:50, Cyrill Gorcunov wrote:
> For some reason we've disabled usage of json formatter in early
> logging since the commit 098324556. Lets allow it back.
>
> v2:
> - add early verification of options for cases like
> box.cfg{log="syslog:", log_format="json"}
> - extend test
>
> v3:
> - pass the whole cfg table into verify_option helper
> - use string.startswith helper in verify_format
>
> branch gorcunov/gh-5121-logger-boot-json-3
> issue https://github.com/tarantool/tarantool/issues/5121
>
> Cyrill Gorcunov (3):
> core/say: allow to use json in boot logger
> lua/log: allow to use json formatter early
> test: app-tap/logger -- test json in boottime logger
>
> src/box/lua/load_cfg.lua | 6 +++---
> src/lib/core/say.c | 7 ++----
> src/lua/log.lua | 42 +++++++++++++++++++++++++++---------
> test/app-tap/logger.test.lua | 19 ++++++++++++----
> 4 files changed, 52 insertions(+), 22 deletions(-)
>
More information about the Tarantool-patches
mailing list