-
Notifications
You must be signed in to change notification settings - Fork 728
feat: tb endpoints for new widgets #3667
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Signed-off-by: Gašper Grom <[email protected]>
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
1 similar comment
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
| required=False, | ||
| ) | ||
| }} | ||
| {% end %} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: Time window uses opened date, not merge
The time bucketing and startDate/endDate filtering use prf.openedAt while the metric is “time to merge”. This makes results depend on when PRs were opened rather than when they merged, so PRs opened in-range but merged later can appear, and PRs opened earlier but merged in-range are excluded, changing the meaning vs the widget being replaced.
Additional Locations (1)
| required=False, | ||
| ) | ||
| }} | ||
| {% end %} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: Time window uses opened date, not review
The time bucketing and startDate/endDate filtering use prf.openedAt while the metric is “time to first review”. This makes results depend on when PRs were opened rather than when they were first reviewed, so PRs opened in-range but reviewed later can appear, and PRs opened earlier but reviewed in-range are excluded, shifting the widget’s meaning.
Note
Add Tinybird pipes for median time to merge/review and Gerrit patchsets per review with KPI and time-series outputs; fix TAGS in review_efficiency.
services/libs/tinybird/pipes/median_time_to_merge.pipegranularity; filters byplatform,startDate,endDate.services/libs/tinybird/pipes/median_time_to_review.pipeservices/libs/tinybird/pipes/patchsets_per_review.pipedataType(median|average), KPI or timeseries; time and repo filters.services/libs/tinybird/pipes/review_efficiency.pipe.Written by Cursor Bugbot for commit ea5c158. This will update automatically on new commits. Configure here.