fix: filter out bad timestamp issue activities to prevent overflows#3693
fix: filter out bad timestamp issue activities to prevent overflows#3693
Conversation
|
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. |
|
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. |
joanagmaia
left a comment
There was a problem hiding this comment.
LGTM ✅
Do you think this might also be an issue for pull requests analysis data? So basically in all widgets where we rely on the timestamps and where we need to make calculations like resolvedAt, mergedAt, shouldn't we always exclude these activities? 🤔
Everywhere else where we have totals we should still include them
We have 1970-dated issue activities that cause overflows when calculating
closedInSecondsand `respondedInSeconds. We now filter these activities out in the issue analysis copy pipeNote
Filters out pre-1971 issue activities in the Tinybird issue analysis pipe to avoid timestamp overflows.
services/libs/tinybird/pipes/issue_analysis_copy_pipe.pipe:toYear(timestamp) >= 1971filters toissues_openedandissues_closedqueries.closedInSecondsandrespondedInSecondsunchanged.TYPEtocopy.Written by Cursor Bugbot for commit 1766184. This will update automatically on new commits. Configure here.