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

There is no way to set socket read timeout #82

Open
jacobuk opened this issue Feb 16, 2019 · 1 comment
Open

There is no way to set socket read timeout #82

jacobuk opened this issue Feb 16, 2019 · 1 comment

Comments

@jacobuk
Copy link

jacobuk commented Feb 16, 2019

There is no way to set a socket read timeout.

socket.setSoTimeout() is not configurable.

So if there are network issues (packet loss for a certain duration) OR if fluentd process itself is not responding then all logging threads could get blocked and bring the application to a standstill.

@komamitsu
Copy link
Member

In fluent-logger-java if we add a new configuration, we need to encode it in a key string of map at https://github.com/fluent/fluent-logger-java/blob/master/src/main/java/org/fluentd/logger/FluentLoggerFactory.java#L56. It's introduced kinda difficulty in terms of maintainability. The key length would get longer and longer if we add new configurations. So we don't have a plan to add new configurations actively.

If you need to ingest data to Fluentd with socket read timeout, you might be interested in another logger https://github.com/komamitsu/fluency#socket-configuration.

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

2 participants