-
Notifications
You must be signed in to change notification settings - Fork 58
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
Cannot use with fluentd-1.7 #77
Comments
ashie
added a commit
to ashie/fluentd
that referenced
this issue
Oct 29, 2019
The pull request fluent#2479 enforces using tzinfo ~> 2.0 but it conflicts with plugins which depend on activesupport such as fluent-plugin-sql since it requires tzinfo ~> 1.1. Until activesupport updgrades it, it would be better to accept also tzinfo 1.x to keep availability of such plugins. Fluentd can accept both versions of tzinfo since fluent#2479 absorb the difference of their API. Fix fluent#2634 See also fluent/fluent-plugin-sql#77 Signed-off-by: Takuro Ashie <[email protected]>
ashie
added a commit
to ashie/fluentd
that referenced
this issue
Oct 29, 2019
The pull request fluent#2479 enforces using tzinfo ~> 2.0 but it conflicts with plugins which depend on activesupport such as fluent-plugin-sql since it requires tzinfo ~> 1.1. Until activesupport updgrades it, it would be better to accept also tzinfo 1.x to keep availability of such plugins. Fluentd can accept both versions of tzinfo since fluent#2479 absorb the difference of their API. Fix fluent#2634 See also fluent/fluent-plugin-sql#77 Signed-off-by: Takuro Ashie <[email protected]>
ashie
added a commit
to ashie/fluentd
that referenced
this issue
Oct 29, 2019
The pull request fluent#2479 enforces using tzinfo ~> 2.0 but it conflicts with plugins which depend on activesupport such as fluent-plugin-sql since it requires tzinfo ~> 1.1. Until activesupport upgrades it, it would be better to accept also tzinfo 1.x to keep availability of such plugins. Fluentd can accept both versions of tzinfo since fluent#2479 absorb the difference of their API. Fix fluent#2634 See also fluent/fluent-plugin-sql#77 Signed-off-by: Takuro Ashie <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
fluent-plugin-sql cannot use with fluentd-1.7 due to conflicting tzinfo versions (as mentioned in fluent/fluentd#2634).
The text was updated successfully, but these errors were encountered: