Skip to content

controllers: report freshness as max(D, S) in milliseconds - #40336

Draft
antiguru wants to merge 3 commits into
freshness/local-delayfrom
freshness/reporting
Draft

antiguru wants to merge 3 commits into
freshness/local-delayfrom
freshness/reporting

Conversation

@antiguru

@antiguru antiguru commented Oct 29, 2026 •

Copy link
Copy Markdown
Member

Behind the reporting flag, the per-minute lag history and the Prometheus metric report max(D, S) in milliseconds instead of rounding up to whole seconds. A REFRESH EVERY view then reads its refresh period instead of 0, and joins of slow sources read their true assignment bound. With the flag off every value is identical to today.

Part of P-CPU-328, design in #40320. Recording and reporting are behind enable_wallclock_freshness_recording and enable_wallclock_freshness_reporting, both off in production, so this changes nothing users see.

🤖 Generated with Claude Code

https://claude.ai/code/session_01LD5gyKhqBoA2DEzJk8jR1R

@antiguru
antiguru added this pull request to stack #40339 October 29, 2026 06:37
@antiguru antiguru changed the title freshness/reporting controllers: report freshness as max(D, S) in milliseconds Oct 29, 2026
@antiguru
antiguru force-pushed the freshness/reporting branch from b1d5223 to 7a8a210 Compare October 29, 2026 07:46
@antiguru
antiguru force-pushed the freshness/reporting branch 3 times, most recently from 86698f2 to 7d892a2 Compare October 29, 2026 20:20
@antiguru
antiguru force-pushed the freshness/reporting branch from 7d892a2 to 3355fb3 Compare October 30, 2026 09:24
@antiguru
antiguru force-pushed the freshness/reporting branch from 3355fb3 to 34293ea Compare October 30, 2026 09:52
antiguru and others added 3 commits October 31, 2026 20:51
@antiguru
antiguru force-pushed the freshness/reporting branch from 34293ea to 8810b76 Compare October 31, 2026 19:53

This branch has not been deployed

No deployments
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