Skip to content

#31: Skip NaN and infinity values #32

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Aug 26, 2019
Merged

#31: Skip NaN and infinity values #32

merged 1 commit into from
Aug 26, 2019

Conversation

bednar
Copy link
Contributor

@bednar bednar commented Aug 26, 2019

Closes #31

Drop NaN and infinity values from fields when writing to InfluxDB

  • CHANGELOG.md updated
  • Rebased/mergeable
  • Tests pass
  • Sign CLA (if not already signed)

@bednar bednar added this to the 1.1.0 milestone Aug 26, 2019
@bednar bednar self-assigned this Aug 26, 2019
@codecov-io
Copy link

Codecov Report

Merging #32 into master will decrease coverage by 17.49%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #32      +/-   ##
=========================================
- Coverage   18.59%    1.1%   -17.5%     
=========================================
  Files          42     307     +265     
  Lines        3931   66292   +62361     
=========================================
  Hits          731     731              
- Misses       3200   65561   +62361
Impacted Files Coverage Δ
Client/Writes/Point.cs 0% <0%> (ø) ⬆️
Client/InfluxDB.Client.Api/Domain/LogEvent.cs 0% <0%> (ø)
...t/InfluxDB.Client.Api/Service/DashboardsService.cs 0% <0%> (ø)
Client/InfluxDB.Client.Api/Domain/TagRule.cs 0% <0%> (ø)
...uxDB.Client.Api/Domain/TelegrafPluginOutputFile.cs 0% <0%> (ø)
...nt/InfluxDB.Client.Api/Domain/OnboardingRequest.cs 0% <0%> (ø)
Client/InfluxDB.Client.Api/Domain/Organization.cs 0% <0%> (ø)
Client/InfluxDB.Client.Api/Domain/Permission.cs 0% <0%> (ø)
...B.Client.Api/Domain/TelegrafPluginInputProcstat.cs 0% <0%> (ø)
Client/InfluxDB.Client.Api/Domain/Views.cs 0% <0%> (ø)
... and 256 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8600be0...c39c7c3. Read the comment docs.

@bednar bednar merged commit 384d79d into master Aug 26, 2019
@bednar bednar deleted the infinity branch August 26, 2019 05:55
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Drop NaN and infinity values from fields when writing to InfluxDB
2 participants