-
Notifications
You must be signed in to change notification settings - Fork 56
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
New metric holder #411
New metric holder #411
Conversation
# Conflicts: # metric/controller.go
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #411 +/- ##
==========================================
+ Coverage 70.16% 70.24% +0.07%
==========================================
Files 112 116 +4
Lines 11838 11845 +7
==========================================
+ Hits 8306 8320 +14
+ Misses 3024 3023 -1
+ Partials 508 502 -6 ☔ View full report in Codecov by Sentry. |
# Conflicts: # pipeline/batch.go # pipeline/pipeline.go
# Conflicts: # fd/file.d.go # pipeline/batch.go # pipeline/pipeline.go # pipeline/processor.go
# Conflicts: # plugin/input/http/http.go # plugin/input/journalctl/journalctl.go # plugin/input/journalctl/reader.go
# Conflicts: # pipeline/pipeline.go # plugin/input/http/http.go
# Conflicts: # pipeline/metrics_holder.go
# Conflicts: # plugin/input/file/file.go # plugin/input/file/provider.go # plugin/output/elasticsearch/elasticsearch.go
03e5619
to
ea087c0
Compare
* Some performance optimizations * Rename RenameReleaser -> MetricDeleter * Benchmark prometheus.MetricVec * Replace parallel benchmark with the loop * Fix string copy * Add tests for the HeldMetric * Fix naming & build (#569) * Fix naming & build * Rollback comment * Update MetricHolder test --------- Co-authored-by: Yaroslav Kirillov <kirillov.y6@gmail.com>
# Conflicts: # plugin/input/kafka/kafka.go
# Conflicts: # plugin/input/file/file.go # plugin/input/file/provider.go # plugin/input/file/worker_test.go
# Conflicts: # plugin/output/kafka/kafka.go
Description
Features:
Fixes #410