Convert buffered output #11
Conversation
Because graphite's minimum metrics time resolution is 10 seconds.
|
|
||
| if @flush_interval < 10 | ||
| log.info("flush_interval less than 10s is not allowed and overwritten to 10s") | ||
| @flush_interval = 10 |
There was a problem hiding this comment.
10 seconds is just a default value for the highest-resolution retention period, and it's configurable, right?
Don't you have any other solution to solve the problem...?
There was a problem hiding this comment.
10 seconds is just a default value for the highest-resolution retention period, and it's configurable, right?
Yes. Users can set arbitrary value greater than 10 seconds with flush_interval parameter in configuration.
Don't you have any other solution to solve the problem...?
An another approach is using this plugin fluent-plugin-bufferize together.
This approach is also acceptable for me.
But it remains some pitfall.
There was a problem hiding this comment.
10 seconds is just a default value for the highest-resolution retention period, and it's configurable, right?
Yes. Users can set arbitrary value greater than 10 seconds with flush_interval parameter in configuration.
Ah, no, I meant that, "isn't it a problem to forcibly set it to 10 seconds even if the setting on the graphite side is not 10 seconds?"
There was a problem hiding this comment.
Ah, no, I meant that, "isn't it a problem to forcibly set it to 10 seconds even if the setting on the graphite side is not 10 seconds?"
Strictly speaking, there is still problem to time resolution.
But current implementation contains more problem.
With current implementation, Fluentd events cannot wait even if 1 second.
Non-Buffered plugin pass through events with no wait time.
Related to #9.
Completely solving #9, we should announce to use aggregating feature plugin like as https://github.com/tagomoris/fluent-plugin-flowcounter in their Fluentd config.