-
Notifications
You must be signed in to change notification settings - Fork 34
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
Enable format/inject plugin helper for Fluentd v0.14 #49
Conversation
* remove config-placeholders / plaintextformatter mixins * add features to support removed mixins: * enable hostname/uuid placeholders by itself (supported %{} style syntax is same with fluent-plugin-s3) * switch to use "out_file" formatter in default, almost compatible with default plaintextformatter configuration * make formatting timezone default in UTC * make formatter/inject plugin helpers enabled to modify/format records in standard way * add compat parameter converter for plaintextformatter for compatibility
@repeatedly Could you review this code? |
@@ -2,7 +2,7 @@ | |||
|
|||
Gem::Specification.new do |gem| | |||
gem.name = "fluent-plugin-webhdfs" | |||
gem.version = "0.5.3" | |||
gem.version = "0.6.0rc1" |
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.
If you update code to work with only v0.14, upgrade major version.
http://docs.fluentd.org/articles/plugin-development#plugin-versioning-policy
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.
Okay.
I don't know the plaintextformatter but others are good for me. |
Let me merge this, and then fix the version number to |
Fixes #48.
This change does:
This plugin will be ready for Fluentd v0.14 after merging this patch.