[Tarantool-patches] [PATCH 0/3] log: allow json formatter in boottime logger
Cyrill Gorcunov
gorcunov at gmail.com
Mon Jun 29 16:37:39 MSK 2020
On Mon, Jun 29, 2020 at 03:50:00PM +0300, Oleg Babin wrote:
> Hi! Thanks for your patch. Looks good but I want to clarify one thing:
>
>
> ```
>
> tarantool> log.cfg{log = 'syslog', format='json'}
> ---
> ...
>
> ```
>
> It works fine but seems such command should throw an error:
>
> "json can't be used with syslog logger", isn't it?
>
> It happened because "ffi.C.log_type()" returns "0" -- boot-time logger.
Good catch! Due to code structure this case is not covered.
I'll update the series and the test. Thanks a lot!!!
More information about the Tarantool-patches
mailing list