Skip to content

Cherry pick PR #8833: cobalt: Refactor histogram fetching process for unit tests#9080

Draft
cobalt-github-releaser-bot wants to merge 1 commit into26.eapfrom
cherry-pick-26.eap-8833
Draft

Cherry pick PR #8833: cobalt: Refactor histogram fetching process for unit tests#9080
cobalt-github-releaser-bot wants to merge 1 commit into26.eapfrom
cherry-pick-26.eap-8833

Conversation

@cobalt-github-releaser-bot
Copy link
Collaborator

Refer to the original PR: #8833

Move histogram fetching and delta calculation logic from H5vccMetricsImpl to a new HistogramFetcher class. This refactoring ensures that histogram data snapshot logic can be tested by unit tests.

Bug: 467384307

Move histogram fetching and delta calculation logic from
H5vccMetricsImpl to a new HistogramFetcher class. This refactoring
ensures that histogram data snapshot logic can be tested by unit tests.

Bug: 467384307
(cherry picked from commit 2236909)
@cobalt-github-releaser-bot
Copy link
Collaborator Author

MERGE CONFLICT CAT

Caution

There were merge conflicts while cherry picking! Check out cherry-pick-26.eap-8833 and fix the conflicts before proceeding. Check the log at https://github.com/youtube/cobalt/actions/runs/21965456058 for details.

@github-actions
Copy link

🤖 Gemini Suggested Commit Message


cobalt: Refactor histogram fetching for unit tests

Move histogram fetching and delta calculation logic from H5vccMetricsImpl
to a new HistogramFetcher class. This refactoring ensures that histogram
data snapshot logic can be tested by unit tests.

Bug: 467384307

💡 Pro Tips for a Better Commit Message:

  1. Influence the Result: Want to change the output? You can write custom prompts or instructions directly in the Pull Request description. The model uses that text to generate the message.
  2. Re-run the Generator: Post a comment with: /generate-commit-message

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants