Skip to content
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

influxdb: don't send zeroes when histogram is empty #414

Merged
merged 1 commit into from
Dec 2, 2016

Conversation

ornicar
Copy link
Contributor

@ornicar ornicar commented Dec 2, 2016

Previous behavior: When a histogram has no records, Kamon sends zeroes.
It's not possible to differenciate between the absence of value, and a value of zero.

New behavior: Kamon doesn't send data for empty histograms.

Previous behavior: When a histogram has no records, Kamon sends zeroes.
It's not possible to differenciate the absence of value from a zero
value.

New behavior: Kamon doesn't send data for empty histograms.
@dpsoft dpsoft added this to the 0.6.4 milestone Dec 2, 2016
@dpsoft
Copy link
Contributor

dpsoft commented Dec 2, 2016

@ornicar thanks for proposing this PR, please sign our CLA we are good to go!

@ornicar
Copy link
Contributor Author

ornicar commented Dec 2, 2016

Meh, CLAs are not fun!

But I really love Kamon so it's the least I can do to thank you guys. Signed.

By the way I'll be deploying this patch to https://lichess.org in 12h. I'll let you know if it goes wrong.

@dpsoft dpsoft merged commit 2ce7b7d into kamon-io:master Dec 2, 2016
@dpsoft
Copy link
Contributor

dpsoft commented Dec 2, 2016

@ornicar done!!!

ornicar added a commit to ornicar/Kamon that referenced this pull request Dec 3, 2016
ivantopo pushed a commit that referenced this pull request Mar 31, 2020
Previous behavior: When a histogram has no records, Kamon sends zeroes.
It's not possible to differenciate the absence of value from a zero
value.

New behavior: Kamon doesn't send data for empty histograms.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants