-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Wrong type for MySQL enum variables with OFF value #7421
Labels
Comments
sjwang90
added
area/mysql
support
Telegraf questions, may be directed to community site or slack
labels
Nov 17, 2020
Hello! I am closing this issue due to inactivity. I hope you were able to resolve your problem, if not please try posting this question in our Community Slack or Community Page. Thank you! |
fxedel
added a commit
to fxedel/telegraf
that referenced
this issue
Jun 21, 2021
Fixes influxdata#8319, influxdata#5711, influxdata#5055, influxdata#7421 See influxdata#5529, influxdata#6624 This could be easily extended to fix influxdata#6671 as well
fxedel
added a commit
to fxedel/telegraf
that referenced
this issue
Jun 21, 2021
Fixes influxdata#8319, influxdata#5711, influxdata#5055, influxdata#7421 See influxdata#5529, influxdata#6624 This could be easily extended to fix influxdata#6671 as well
sspaink
removed
the
support
Telegraf questions, may be directed to community site or slack
label
Jul 1, 2021
Fixed with #9403 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Relevant telegraf.conf:
System info:
Telegraf 1.14.0 (git: HEAD fefd7ff)
Debian GNU/Linux 9 (stretch)
Description
Some MySQL variables, like transaction_write_set_extraction, allow for multiple string values including
OFF
, e.g.:OFF
,MURMUR32
orXXHASH64
.I expected these variables to be gathered as string, however, the Off value is converted to the integer 0 in ParseValue. This conversion is good for simple On/Off variables, but doesn't fit for enumerations like transaction_write_set_extraction as it produces errors like these:
Proposed fix
I don't know what the best solution would be; maybe a hardcoded list of variables that should parse On/Off as string?
The text was updated successfully, but these errors were encountered: