-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
input_chunk: change mem buf overlimit message to warning #1909
Conversation
Signed-off-by: Wesley Pettit <[email protected]>
205760d
to
16778d1
Compare
thanks |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this the fix seeing only log level is changed from debug to warn
@CoviD2 Yeah this isn't a full fix, just an improvement in log messaging.. |
It will be available to the latest 1.4+ versions right? |
Can we also alter https://github.com/PettitWesley/fluent-bit/blob/16778d1bce48339eebcc38c081c6f4909ff3ff05/src/flb_input_chunk.c#L295? if the pause message is a WARN so should the resume should also be an WARN. |
@ts3ng How about info level? |
@PettitWesley As long as the levels are the same for the pause and the resume. I'm fine with INFO or WARN. |
kubernetes v1.23.4
中也有这样的问题,出现
这样的日志后,就不会在往kafka发送数据 |
See #1903, I think this message should probably be a warning instead of debug.