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

Add MetricRecorder implementation #11128

Merged
merged 7 commits into from
Apr 26, 2024
Merged

Add MetricRecorder implementation #11128

merged 7 commits into from
Apr 26, 2024

Commits on Apr 25, 2024

  1. added MetricRecordedImpl and unite tests for MetricInstrumentRegistry

    @DNVindhya
    DNVindhya committed Apr 25, 2024
    Configuration menu
    Copy the full SHA
    413393a View commit details
    Browse the repository at this point in the history

  2. changed MetricRcorderImpl to public

    @DNVindhya
    DNVindhya committed Apr 25, 2024
    Configuration menu
    Copy the full SHA
    295e58d View commit details
    Browse the repository at this point in the history

  3. updated MetricInstrumenetRegistry to use array instead of ArrayList; … 

    …addressed review comments
    @DNVindhya
    DNVindhya committed Apr 25, 2024
    Configuration menu
    Copy the full SHA
    0f95f8e View commit details
    Browse the repository at this point in the history

Commits on Apr 26, 2024

  1. Renamed record<>Counter APIs to add<>Counter. Added check for mismatc… 

    …hed label values
    @DNVindhya
    DNVindhya committed Apr 26, 2024
    Configuration menu
    Copy the full SHA
    7167dcb View commit details
    Browse the repository at this point in the history

  2. addressed review comments

    @DNVindhya
    DNVindhya committed Apr 26, 2024
    Configuration menu
    Copy the full SHA
    677f82c View commit details
    Browse the repository at this point in the history

  3. updated to use array length

    @DNVindhya
    DNVindhya committed Apr 26, 2024
    Configuration menu
    Copy the full SHA
    2dbf553 View commit details
    Browse the repository at this point in the history

  4. added lock for instruments array

    @DNVindhya
    DNVindhya committed Apr 26, 2024
    Configuration menu
    Copy the full SHA
    5f98cdf View commit details
    Browse the repository at this point in the history