Skip to content

feat(dispatch): expose readonly interfaces#4679

Draft
siavashs wants to merge 3 commits intoprometheus:mainfrom
siavashs:feat/ag-interface
Draft

feat(dispatch): expose readonly interfaces#4679
siavashs wants to merge 3 commits intoprometheus:mainfrom
siavashs:feat/ag-interface

Conversation

@siavashs
Copy link
Contributor

@siavashs siavashs commented Nov 3, 2025

Groups() method in dispatcher does extensive data manipulation to
staisfy the API. Instead it can return a snapshot of its internal state
and let the API handle the data processing.

  • expose readonly interfaces for AggregationGroup and Route
  • move Groups logic out of dispatcher and into the api handler

Signed-off-by: Ethan Hunter <ehunter@hudson-trading.com>
Signed-off-by: Ethan Hunter <ehunter@hudson-trading.com>
@siavashs siavashs force-pushed the feat/ag-interface branch 4 times, most recently from 02ee92f to f5cf518 Compare November 4, 2025 16:22
Groups() method in dispatcher does extensive data manipulation to
staisfy the API. Instead it can return a snapshot of its internal state
and let the API handle the data processing.

- expose readonly interfaces for AggregationGroup and Route
- move Groups logic out of dispatcher and into the api handler

Signed-off-by: Siavash Safi <siavash@cloudflare.com>
@siavashs siavashs self-assigned this Nov 14, 2025
@github-actions
Copy link

github-actions bot commented Feb 6, 2026

This pull request is stale because it has been open 30 days with no activity.

@github-actions github-actions bot added the stale label Feb 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants