Skip to content

fix(publishing/influx_publisher): use float64 for numeric field types#46

Merged
stakach merged 1 commit intomasterfrom
limit-fields-to-float64-values
Oct 18, 2022
Merged

fix(publishing/influx_publisher): use float64 for numeric field types#46
stakach merged 1 commit intomasterfrom
limit-fields-to-float64-values

Conversation

@stakach
Copy link
Copy Markdown
Member

@stakach stakach commented Oct 18, 2022

avoids issues with JSON ambiguity and is the default influxdb type

avoids issues with JSON ambiguity and is the default influxdb type
@stakach stakach requested a review from w-le October 18, 2022 02:10
@stakach stakach self-assigned this Oct 18, 2022
@github-actions github-actions bot added the type: bug something isn't working label Oct 18, 2022
@stakach
Copy link
Copy Markdown
Member Author

stakach commented Oct 18, 2022

@w-le this might cause issues with any existing integer types
I found this thread on how to convert any existing types
https://community.influxdata.com/t/chage-field-type-from-string-to-float/16222/15

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: bug something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant