Skip to content

observed value should be stored the bucket - #176

Merged
dmagliola merged 1 commit into
prometheus:masterfrom
ganmacs:fix-bug-histgram-range
Jan 28, 2020
Merged

observed value should be stored the bucket#176
dmagliola merged 1 commit into
prometheus:masterfrom
ganmacs:fix-bug-histgram-range

Conversation

@ganmacs

@ganmacs ganmacs commented Jan 8, 2020

Copy link
Copy Markdown
Contributor

whose value is the same as the observed value

Seems to be working in v0.9.0

self[bucket] += 1 if value <= bucket

which behavior is correct?

whose value is same as the observed value

Signed-off-by: Yuta Iwama <ganmacs@gmail.com>
@ganmacs
ganmacs force-pushed the fix-bug-histgram-range branch from 7550f3b to c931a04 Compare January 8, 2020 08:59
@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 100.0% when pulling 7550f3b on ganmacs:fix-bug-histgram-range into 260e51a on prometheus:master.

1 similar comment
@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 100.0% when pulling 7550f3b on ganmacs:fix-bug-histgram-range into 260e51a on prometheus:master.

@coveralls

coveralls commented Jan 8, 2020

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 100.0% when pulling c931a04 on ganmacs:fix-bug-histgram-range into 260e51a on prometheus:master.

@dmagliola

Copy link
Copy Markdown
Collaborator

Hello @ganmacs. Sorry for the very late reply!
Thank you for sending this fix.
While this is a fix, this is a breaking change, so we'll be cutting a new version of the API soon with this included.
Sorry again it took so long to get back

@dmagliola
dmagliola merged commit 56bd577 into prometheus:master Jan 28, 2020
@ganmacs
ganmacs deleted the fix-bug-histgram-range branch January 28, 2020 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants