Fix behaviour for all metric types when only calls with labels are made #81
Conversation
|
@MrLotU I understand that this PR has been open for a while, and it appears that some fixes were merged in the past. Nevertheless, I wanted to reach out and inquire if there’s anything else you believe we could improve upon. Additionally, there’s been a significant change in the code base recently. Thank you! |
|
Hey @incertum, I'm honestly not quite sure why this PR was not merged when it was opened 3 years ago. I'm no longer actively involved with the project, so happy for this to be merged if the original issues still need to be addressed, and this PR is still a fair solution with all the evolution this project has gone through since the opening of this PR. Also more than happy to have it closed if the original issues have been solved in some different way, or if the implementation in this PR is no longer in line with the projects current setup. |
Please accept our apologies for not having reviewed this PR. I’ll check this thoroughly and reply next week. |
|
Nono, absolutely no worries. I'm 99% sure this PR was opened by me when I was still actively maintaining this project, and just had it somehow completely fall of my radar |
Fixes #80 #77
Checklist
Fixes an internal inconsistency where recording to a summary or histogram with labels would overflow into their main (no labels) versions too. (#77)
Also removes "empty" reports of metrics (#80)