Skip to content

Consolidate metrics emit helpers#43

Open
forge33 wants to merge 1 commit into
masterfrom
pb/consolidate_metrics_emitters
Open

Consolidate metrics emit helpers#43
forge33 wants to merge 1 commit into
masterfrom
pb/consolidate_metrics_emitters

Conversation

@forge33
Copy link
Copy Markdown

@forge33 forge33 commented Jun 1, 2026

Related issue: github#1672

Description

  • Move the small Emit... metric helpers into a single go/metrics/emit.go file.
  • Move the Go runtime reporter into the same consolidated metrics file.
  • Move related tests into go/metrics/emit_test.go.

🎩

Tophatted as part of #42 #45 #46 #47

In case this PR introduced Go code changes:

  • contributed code is using same conventions as original code
  • script/cibuild returns with no formatting errors, build errors or unit test errors.

@forge33 forge33 force-pushed the pb/consolidate_metrics_emitters branch 2 times, most recently from aa3774f to 63e4992 Compare June 1, 2026 18:30
@forge33 forge33 added the #gsd:50633 Data Storage: gh-ost Observability Instrumentation label Jun 1, 2026
@forge33 forge33 requested a review from grodowski June 3, 2026 13:08
@forge33 forge33 force-pushed the pb/consolidate_metrics_emitters branch from 63e4992 to bd9bdb1 Compare June 3, 2026 14:40
@forge33 forge33 changed the base branch from pb/refactor_metrics_emitter to master June 3, 2026 14:40
Keep the small metric emission helpers, runtime reporter, and their tests together so the metrics package has fewer one-function files.
@forge33 forge33 force-pushed the pb/consolidate_metrics_emitters branch from bd9bdb1 to c4e2b06 Compare June 3, 2026 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

#gsd:50633 Data Storage: gh-ost Observability Instrumentation to-upstream

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants