Skip to content

fix: members_sorted schedule#4376

Open
epipav wants to merge 2 commits into
mainfrom
fix/reschedule-members-sorted-copy
Open

fix: members_sorted schedule#4376
epipav wants to merge 2 commits into
mainfrom
fix/reschedule-members-sorted-copy

Conversation

@epipav

@epipav epipav commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

Signed-off-by: anilb <epipav@gmail.com>
Copilot AI review requested due to automatic review settings July 22, 2026 09:36
@github-actions

Copy link
Copy Markdown
Contributor

⚠️ Jira Issue Key Missing

Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability.

Example:

  • feat: add user authentication (CM-123)
  • feat: add user authentication (IN-123)

Projects:

  • CM: Community Data Platform
  • IN: Insights

Please add a Jira issue key to your PR title.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Conventional Commits FTW!

@cursor

cursor Bot commented Jul 22, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Single cron timing tweak with no query changes; the main risk is hourly ordering/freshness vs other staggered copy pipes if the new minute was chosen incorrectly.

Overview
Updates COPY_SCHEDULE on members_sorted_copy_pipe.pipe from 8 * * * * to 36 * * * *, so the replace copy into members_sorted runs later each hour instead of at :08.

No SQL or datasource logic changes—only when the filtered member snapshot (used downstream e.g. memberId IN (SELECT id FROM members_sorted) in activityRelations bucket enrich copies) is refreshed.

Reviewed by Cursor Bugbot for commit 79a057a. Bugbot is set up for automated code reviews on this repo. Configure here.

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Moves the hourly members_sorted Tinybird refresh from minute 8 to minute 36.

Changes:

  • Updates the copy schedule to 36 * * * *.
  • PR title is missing the required (CM-XXX) JIRA key.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@epipav epipav changed the title fix: members_sorted schedule fix: members_sorted schedule Jul 22, 2026
@epipav
epipav requested a review from gaspergrom July 22, 2026 09:41
Copilot AI review requested due to automatic review settings July 22, 2026 10:29

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

TARGET_DATASOURCE members_sorted
COPY_MODE replace
COPY_SCHEDULE 8 * * * *
COPY_SCHEDULE 36 * * * *
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.

3 participants