Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Remove histogram::Histogram from public API #935

Merged
merged 1 commit into from
Feb 16, 2024
Merged

Remove histogram::Histogram from public API #935

merged 1 commit into from
Feb 16, 2024

Commits on Feb 15, 2024

  1. metrics: remove Histogram from MetricsError 

    Public MetricsError enum contains a reference to `Histogram`.

In fact, the `MetricsError::Poison` is never really constructed
since we always unwrap the PoisonError when trying to acquire
the mutex.

This is why we get rid of the `MetricsError::Poison` enum value and
convert the `MetricsError` enum to struct holding an error cause.
    @muzarski
    muzarski committed Feb 15, 2024
    Configuration menu
    Copy the full SHA
    caa559c View commit details
    Browse the repository at this point in the history