Skip to content

feat(metrics): emit Rust metrics through metrics-rs - #566

Closed
lvthanh03 wants to merge 1 commit into
mainfrom
tony/metrics-rs-facade
Closed

feat(metrics): emit Rust metrics through metrics-rs#566
lvthanh03 wants to merge 1 commit into
mainfrom
tony/metrics-rs-facade

Conversation

@lvthanh03

@lvthanh03 lvthanh03 commented Sep 1, 2026

Copy link
Copy Markdown
Member

Summary

  • emit sentry_arroyo Rust counters, gauges, and timers through the metrics-rs facade
  • use the single recorder/exporter installed by the consuming application instead of a separate Arroyo metrics backend
  • preserve existing metric names, labels, and millisecond timer values
  • adapt the legacy StatsdRecorder API to the same metrics-rs global for backward compatibility

Applications own metrics initialization and install exactly one metrics-rs recorder/exporter. Arroyo only emits through that recorder and does not initialize or operate a separate exporter.

Test plan

  • cargo fmt --all --check
  • cargo check
  • cargo test --test metrics_facade --test legacy_metrics_recorder
  • cargo test --doc
  • cargo test --all-targets with the six local-Kafka-dependent tests skipped (54 passed; CI provisions Kafka for the full suite)

@lvthanh03
lvthanh03 requested review from a team as code owners September 1, 2026 19:52
@lvthanh03
lvthanh03 marked this pull request as draft September 1, 2026 19:53
@lvthanh03 lvthanh03 closed this Sep 1, 2026
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.

1 participant