Daijiro Fukuda

Results 519 comments of Daijiro Fukuda

I confirmed the following PR fixed this issue. * #4474

> I confirmed the following PR fixed this issue. > > * #4474 Oh... this issue still remains in v1.16.x because it does not include #4474.

> only backport is pending #4474 is a large fix and has some specification changes, so it is not preferable to backport it to v1.16. It is a difficult problem...

I'm not sure yet, but it might be a good idea to backport just this commit about `parser_msgpack` to v1.16. * https://github.com/fluent/fluentd/pull/4474/commits/8433ca7ff78525bf89e2fea0f58a048ff1e2b6a9 Or, as @ashie says in https://github.com/fluent/fluentd/issues/4346#issuecomment-1846623604, it might...

I'm sorry. This breaks Windows `--daemon` feature since v1.16.0. * #4065 This should be fixed.

From #4065 > `Supervisor.load_config()` is called in ServerEngine's reloading function, but Fluentd doesn't use the function even when `SIGHUP` or `SIGUSR2`. So I can't see the reason for initializing the...

I'm sorry. This change breaks the specification. https://github.com/fluent/fluentd/pull/4064/files#diff-fe3c9c8a2fe3872b84e0de1d14ecd669258b0e57609c470b601bdfabedb91224R49 This should be fixed.

@mrudrego Thanks! Sorry for my late response. Currently, I'm busy on checking the issues related to #3614, so my response may be slow for a while. Sorry. The fix for...

Fixed by #4605. It will be released on v1.16.6 and v1.18.0 (or possibly v1.17.2 (undecided))