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

stats: add support for multiple stats handlers in a single client or server #5347

Merged
merged 6 commits into from Jun 3, 2022
Merged

stats: add support for multiple stats handlers in a single client or server #5347

merged 6 commits into from Jun 3, 2022

Commits on May 12, 2022

  1. stats: Support multiple StatsHandlers

    @lidizheng
    lidizheng committed May 12, 2022
    Copy the full SHA
    6868f8b View commit details
    Browse the repository at this point in the history

  2. Fix a side-impact to channelz

    @lidizheng
    lidizheng committed May 12, 2022
    Copy the full SHA
    423d741 View commit details
    Browse the repository at this point in the history

Commits on Jun 1, 2022

  1. Adopt reviewer's suggestions: 

    1. Pluralize the variable names for slices
2. Remove the unnecessary ifs
3. Simplify the unit test
    @lidizheng
    lidizheng committed Jun 1, 2022
    Copy the full SHA
    e78e211 View commit details
    Browse the repository at this point in the history

Commits on Jun 2, 2022

  1. Stablize stats_test

    @lidizheng
    lidizheng committed Jun 2, 2022
    Copy the full SHA
    b7e4cf2 View commit details
    Browse the repository at this point in the history

  2. Add the "shs" variable

    @lidizheng
    lidizheng committed Jun 2, 2022
    Copy the full SHA
    821f1f9 View commit details
    Browse the repository at this point in the history

  3. Add one more shs variable

    @lidizheng
    lidizheng committed Jun 2, 2022
    Copy the full SHA
    3ceb045 View commit details
    Browse the repository at this point in the history