[WIP] Add 14-day time bound to failed-tests matviews#3651
Conversation
|
Pipeline controller notification For optional jobs, comment This repository is configured in: automatic mode |
|
Skipping CI for Draft Pull Request. |
|
Warning Review limit reached
More reviews will be available in 15 minutes and 5 seconds. Learn how PR review limits work. Your organization has used up its prepaid credits, and credit purchases are no longer available. Enable the review add-on in the billing tab to keep reviews running — you're only billed for reviews past your plan's rate limits ($0.25/file). ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based credits. 🚦 How do rate limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan refill rate. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, the refill rate gradually slows as usage increases. The highest same-day bursts are limited more strictly. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Repository YAML (base), Central YAML (inherited) Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (2)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: mstaeble The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
bfdb21f to
1444997
Compare
The prow_job_failed_tests_by_day and prow_job_failed_tests_by_hour materialized views scanned the entire prow_job_run_tests table with no time filter, taking ~3 minutes per refresh. The only consumer (job analysis page) uses a 14-day window. Adding a 14-day time bound enables RANGE partition pruning within each release, reducing refresh time to ~1 minute and row count from 1.28M to 133K. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1444997 to
d6528a0
Compare
Summary
prow_job_failed_tests_by_dayandprow_job_failed_tests_by_hourmaterialized views scanned the entireprow_job_run_teststable with no time filter, taking ~3 minutes per refresh. The only consumer (job analysis page) uses a 14-day window.Benchmarks (staging,
enable_partitionwise_join = on)Matview read performance is unchanged (~46ms).
Alternative
PR #3647 removes the matviews entirely and queries the partitioned table directly (~249ms warm reads, no refresh needed). This PR is a smaller, lower-risk change that keeps the matview pattern.
Test plan
go build ./...go vetclean🤖 Generated with Claude Code