fluent / fluent-plugin-s3

Amazon S3 input and output plugin for Fluentd
https://docs.fluentd.org/output/s3
314 stars 218 forks source link

Fix documentation with invalid include_time_key setting #310

Closed kanga333 closed 4 years ago

kanga333 commented 4 years ago

JSON formatter plugin does not support include_time_key and time_key. Inject section can be used to include the time_key.

If you actually specify v0.12 style include_time_key in s3_output of v1, it will be replaced by inject section.

kanga333 commented 4 years ago

include_time_key in the format section is an invalid setting. include_time_key at the top level is a deprecated but valid setting.

I want to fix the wrong example of include_time_key setting in v1 style.

ganmacs commented 4 years ago

ah, I finally got your point. LGTM. thank you!