Skip to content

fix: share cdc#3872

Draft
AmarTrebinjac wants to merge 1 commit into
mainfrom
fix-cdc-for-share-ach
Draft

fix: share cdc#3872
AmarTrebinjac wants to merge 1 commit into
mainfrom
fix-cdc-for-share-ach

Conversation

@AmarTrebinjac
Copy link
Copy Markdown
Contributor

No description provided.

@pulumi
Copy link
Copy Markdown

pulumi Bot commented May 13, 2026

🍹 The Update (preview) for dailydotdev/api/prod (at 82289c0) was successful.

✨ Neo Explanation

Routine deployment of a bug fix for share-click achievement tracking, with standard migration jobs cycling to the new commit. ✅ Low Risk

This is a routine application deployment rolling out commit 82289c03 (previously c0d46faa) across all Kubernetes workloads. The code change fixes the ShareClickMilestone and SharePostsClicked achievement progress logic: rather than passing the raw per-event click count to ShareClickMilestone, it now queries the user-level max clicks across all their share posts in a single combined query, and treats both event types as absolute-value (non-accumulating) progress. The DB and Clickhouse migration jobs are recycled as expected with each new commit hash.

🔵 Info — The new combined query (MAX(pa.clicks) + COUNT(*) FILTER (WHERE pa.clicks > 0)) executes on every post_analytics CDC event for share posts. Depending on volume, this adds one extra DB round-trip per event compared to the previous two-query approach, though the consolidation is a net improvement. No schema changes are implied by the logic diff, but the presence of new DB and Clickhouse migration jobs confirms there are accompanying migrations — verify their contents are backward-compatible before the new pods go live.

Resource Changes

    Name                                                       Type                           Operation
~   vpc-native-post-analytics-clickhouse-cron                  kubernetes:batch/v1:CronJob    update
-   vpc-native-api-db-migration-c0d46faa                       kubernetes:batch/v1:Job        delete
~   vpc-native-calculate-top-readers-cron                      kubernetes:batch/v1:CronJob    update
~   vpc-native-daily-digest-cron                               kubernetes:batch/v1:CronJob    update
~   vpc-native-sync-subscription-with-cio-cron                 kubernetes:batch/v1:CronJob    update
~   vpc-native-worker-job-deployment                           kubernetes:apps/v1:Deployment  update
~   vpc-native-temporal-deployment                             kubernetes:apps/v1:Deployment  update
~   vpc-native-update-tag-materialized-views-cron              kubernetes:batch/v1:CronJob    update
~   vpc-native-hourly-notification-cron                        kubernetes:batch/v1:CronJob    update
~   vpc-native-user-profile-analytics-clickhouse-cron          kubernetes:batch/v1:CronJob    update
~   vpc-native-clean-channel-highlights-cron                   kubernetes:batch/v1:CronJob    update
~   vpc-native-generic-referral-reminder-cron                  kubernetes:batch/v1:CronJob    update
~   vpc-native-update-achievement-rarity-cron                  kubernetes:batch/v1:CronJob    update
~   vpc-native-materialize-monthly-best-post-archives-cron     kubernetes:batch/v1:CronJob    update
+   vpc-native-api-db-migration-9f90231a                       kubernetes:batch/v1:Job        create
~   vpc-native-personalized-digest-cron                        kubernetes:batch/v1:CronJob    update
~   vpc-native-personalized-digest-deployment                  kubernetes:apps/v1:Deployment  update
~   vpc-native-generate-search-invites-cron                    kubernetes:batch/v1:CronJob    update
~   vpc-native-update-source-public-threshold-cron             kubernetes:batch/v1:CronJob    update
~   vpc-native-update-current-streak-cron                      kubernetes:batch/v1:CronJob    update
~   vpc-native-post-analytics-history-day-clickhouse-cron      kubernetes:batch/v1:CronJob    update
+   vpc-native-api-clickhouse-migration-9f90231a               kubernetes:batch/v1:Job        create
~   vpc-native-update-trending-cron                            kubernetes:batch/v1:CronJob    update
~   vpc-native-rotate-daily-quests-cron                        kubernetes:batch/v1:CronJob    update
~   vpc-native-check-analytics-report-cron                     kubernetes:batch/v1:CronJob    update
~   vpc-native-user-profile-analytics-history-clickhouse-cron  kubernetes:batch/v1:CronJob    update
~   vpc-native-rotate-weekly-quests-cron                       kubernetes:batch/v1:CronJob    update
~   vpc-native-validate-active-users-cron                      kubernetes:batch/v1:CronJob    update
~   vpc-native-clean-zombie-user-companies-cron                kubernetes:batch/v1:CronJob    update
~   vpc-native-channel-digests-cron                            kubernetes:batch/v1:CronJob    update
~   vpc-native-ws-deployment                                   kubernetes:apps/v1:Deployment  update
~   vpc-native-clean-gifted-plus-cron                          kubernetes:batch/v1:CronJob    update
~   vpc-native-user-profile-updated-sync-cron                  kubernetes:batch/v1:CronJob    update
~   vpc-native-expire-super-agent-trial-cron                   kubernetes:batch/v1:CronJob    update
~   vpc-native-update-views-cron                               kubernetes:batch/v1:CronJob    update
~   vpc-native-materialize-yearly-best-post-archives-cron      kubernetes:batch/v1:CronJob    update
~   vpc-native-user-posts-analytics-refresh-cron               kubernetes:batch/v1:CronJob    update
~   vpc-native-update-highlighted-views-cron                   kubernetes:batch/v1:CronJob    update
~   vpc-native-update-tags-str-cron                            kubernetes:batch/v1:CronJob    update
~   vpc-native-channel-highlights-cron                         kubernetes:batch/v1:CronJob    update
~   vpc-native-clean-zombie-opportunities-cron                 kubernetes:batch/v1:CronJob    update
... and 11 other changes

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