client_rust
client_rust copied to clipboard
Clone Family.metrics and iterate over family
Problems
-
There is no way to copy/clone the
HashMap<S,M>
within family.- Use case: Storing metrics to compare against later
-
There is no way to iterate over a metric
Family
or get metrics without knowing the labels.- The only to get all metrics is to manually call
get_or_create()
for every metric. This isn't great if you don't know the label names (don't store them etc...).
- The only to get all metrics is to manually call
I am happy to tackle these if you want. I think making M
clone (not sure why it isn't atm) and returning a new hashmap with the cloned values would fix both these problems.
Can you expand on the use-cases that you want to solve with the above proposed changes?
Sure.
Iter
The primary problem I have found when using this library is that without knowing all the label keys, it's not possible (please let me know if it is) to get any metrics.
Suppose I have an application that requires me to spawn n
components to process some workload. At runtime each of these will be spawned and assigned a uuid. In order to monitor these I would like to attach a metrics counter to each (using uuid as the primary label field). I will create a Family<ComponentLabel, Counter>
for this.
Now suppose I am now in some other part of the application and would like to iterate over every metric in that family. The only way to do this at the moment (i believe) would be to store a vec of uuids that you inserted into the family and manually iterate over each and get_or_create()
.
The family "hashmap" is already storing all the data we need to iterate over it, but doesn't expose any functionality to interact with it (outside of get_or_create
). Requiring a separate data structure to store insertions for later seems wasteful and unessesary.
Clone
Using the above example, suppose I want to compare metrics of each component over time. To do this, I would like to take a snapshot of the metric family at time t
and then some time later in the processing take another snapshot and then compare the two. This may be used for example to see if the processing is lagging (counters are not increasing).
This is not possible at the moment as M
(in `Family<S,M>) is not clone and the underlying family hashmap isn't exposed.
Suggestion
To solve both these problems I suggest making M
clone (in Family<S,M>
) and exposing the read method of family.metrics
.
E.g. in metrics/family.rs
:
pub fn read(&self) -> RwLockReadGuard<HashMap<S, M>> {
self.metrics.read()
}
This would solve both these problems and allow a user to do something like:
family .read().iter() .map(|(label, counter)| { do_something() })
Also providing a snapshot()
method on the registry,family or metrics would be nice. This should be easy to implement and give user's a nice interface to compare metrics over time.
I am also looking for these features.
My use case is to expose some metrics and labels, via a proprietary endpoint in addition to the standard /metrics
endpoint. Currently I have to store the inserted labels in a HashSet
which seems unnecesary.