-
Notifications
You must be signed in to change notification settings - Fork 64
go.mod: upgrade Prometheus #567
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
Conversation
c851403
to
32d0780
Compare
Upgrade the Prometheus library and the corresponding code. Signed-off-by: Giedrius Statkevičius <[email protected]>
2fb5d1d
to
343c7c2
Compare
Signed-off-by: Giedrius Statkevičius <[email protected]>
Somehow
Need to investigate & fix. |
Signed-off-by: Giedrius Statkevičius <[email protected]>
Now its much better.
I can take a look at them today. |
#582 should fix double_exponential_smoothing. |
In my recent bench runs, things are looking good.
The regression in subquery is because of the additional iteration inside sum_over_time to check for mixed samples. promql-engine/ringbuffer/functions.go Line 169 in f181edf
|
Signed-off-by: 🌲 Harry 🌊 John 🏔 <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I hope we get stability in Prometheus soon 🤞
Upgrade the Prometheus library and the corresponding code.