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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants