Custom Collector with `sub_registry_with_prefix` outputs name multiple times about client_rust

[mxinden]

You are right. Thank you for the report. I have discovered the same with https://github.com/libp2p/rust-libp2p/tree/master/misc/metrics.

I have fixed this issue in #149 (not yet released). A nice side-effect is that it significantly simplifies the Collector trait.

@allada would you mind testing with latest master, i.e. with #149 included?

I am sorry for not having patched this earlier. Thanks for debugging.

(Good to see the Collector trait, while faulty, needed and used.)

[allada]

It looks like the entire API has changed a lot and without having non-code-based documentation that I can use it's going to be difficult to test this in our system without refactoring everything.

However, you might be interested in seeing the wrappers we are using, as it might give you some ideas on API designs:

Internal metrics api:
https://github.com/allada/turbo-cache/blob/fd1de941b796287abe24dad852ed50a27d5d86af/util/prometheus_utils.rs

Example usage:
https://github.com/allada/turbo-cache/blob/fd1de941b796287abe24dad852ed50a27d5d86af/cas/worker/running_actions_manager.rs#L1138
https://github.com/allada/turbo-cache/blob/fd1de941b796287abe24dad852ed50a27d5d86af/cas/worker/local_worker.rs#L484
https://github.com/allada/turbo-cache/blob/fd1de941b796287abe24dad852ed50a27d5d86af/util/evicting_map.rs#L375

There was also a short discussion that we had here:
TraceMachina/nativelink#209

You might be interested in the idea of a procedural macro.

[mxinden]

It looks like the entire API has changed a lot and without having non-code-based documentation that I can use it's going to be difficult to test this in our system without refactoring everything.

I am sorry for the trouble this causes on your end. I am not sure I am fully following. What is non-code-based documentation? How can I make your transition to the new Collector trait easier @allada?

You might be interested in the idea of a procedural macro.

I am not sure how a procedural macro could simplify the Collector trait. Do you have concrete suggestions in mind? In general, I try to use macros as a last resort only, especially when exposed on the public API surface.