Skip to content
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

$fluent->errors stocks all error logs unlimitedly. #16

Open
tokuhirom opened this issue Dec 20, 2016 · 1 comment
Open

$fluent->errors stocks all error logs unlimitedly. #16

tokuhirom opened this issue Dec 20, 2016 · 1 comment

Comments

@tokuhirom
Copy link
Contributor

Current implementation stores error logs in $fluent->errors unlimitedly. It would cause OOM killer if a "td-agent" process were gone(by segmentation fault, oom killer, etc.)
I know, fluent-logger-perl users should clear $fluent->errors buffer after sending a message. But, this issue is easily happen in a long running process, and hard to detect.

I suggest to add the hard limit to $fluent->errors.

@tokuhirom
Copy link
Contributor Author

connect_error_history has same issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant