Skip to content

Conversation

@idoshamun
Copy link
Member

No description provided.

@pulumi
Copy link

pulumi bot commented Dec 28, 2025

🍹 The Update (preview) for dailydotdev/api/prod (at 6d5e839) was successful.

Resource Changes

    Name                                                   Type                                      Operation
~   vpc-native-clean-stale-user-transactions-cron          kubernetes:batch/v1:CronJob               update
~   vpc-native-update-tag-recommendations-cron             kubernetes:batch/v1:CronJob               update
+   vpc-native-api-clickhouse-migration-097b5533           kubernetes:batch/v1:Job                   create
~   vpc-native-personalized-digest-cron                    kubernetes:batch/v1:CronJob               update
~   vpc-native-user-profile-updated-sync-cron              kubernetes:batch/v1:CronJob               update
~   vpc-native-temporal-deployment                         kubernetes:apps/v1:Deployment             update
~   api-redis                                              dailydotdev:pulumi-common:Redis           update
~   vpc-native-update-highlighted-views-cron               kubernetes:batch/v1:CronJob               update
-   vpc-native-api-clickhouse-migration-f7c3cac3           kubernetes:batch/v1:Job                   delete
-   vpc-native-api-db-migration-f7c3cac3                   kubernetes:batch/v1:Job                   delete
~   vpc-native-sync-subscription-with-cio-cron             kubernetes:batch/v1:CronJob               update
~   vpc-native-daily-digest-cron                           kubernetes:batch/v1:CronJob               update
~   vpc-native-update-views-cron                           kubernetes:batch/v1:CronJob               update
~   vpc-native-clean-zombie-user-companies-cron            kubernetes:batch/v1:CronJob               update
~   vpc-native-private-deployment                          kubernetes:apps/v1:Deployment             update
~   vpc-native-update-trending-cron                        kubernetes:batch/v1:CronJob               update
~   vpc-native-update-source-tag-view-cron                 kubernetes:batch/v1:CronJob               update
~   vpc-native-calculate-top-readers-cron                  kubernetes:batch/v1:CronJob               update
~   vpc-native-hourly-notification-cron                    kubernetes:batch/v1:CronJob               update
~   vpc-native-check-analytics-report-cron                 kubernetes:batch/v1:CronJob               update
~   vpc-native-generic-referral-reminder-cron              kubernetes:batch/v1:CronJob               update
~   vpc-native-update-source-public-threshold-cron         kubernetes:batch/v1:CronJob               update
~   vpc-native-clean-gifted-plus-cron                      kubernetes:batch/v1:CronJob               update
~   vpc-native-generate-search-invites-cron                kubernetes:batch/v1:CronJob               update
~   vpc-native                                             pulumi:providers:kubernetes               update
~   vpc-native-personalized-digest-deployment              kubernetes:apps/v1:Deployment             update
~   vpc-native-bg-deployment                               kubernetes:apps/v1:Deployment             update
+   vpc-native-api-db-migration-097b5533                   kubernetes:batch/v1:Job                   create
~   vpc-native-clean-zombie-images-cron                    kubernetes:batch/v1:CronJob               update
~   vpc-native-clean-zombie-users-cron                     kubernetes:batch/v1:CronJob               update
~   vpc-native-clean-zombie-opportunities-cron             kubernetes:batch/v1:CronJob               update
~   clickhouse-sync                                        dailydotdev:pulumi-common:ClickHouseSync  update
~   vpc-native-update-tags-str-cron                        kubernetes:batch/v1:CronJob               update
~   vpc-native-deployment                                  kubernetes:apps/v1:Deployment             update
~   vpc-native-post-analytics-clickhouse-cron              kubernetes:batch/v1:CronJob               update
~   vpc-native-post-analytics-history-day-clickhouse-cron  kubernetes:batch/v1:CronJob               update
~   vpc-native-ws-deployment                               kubernetes:apps/v1:Deployment             update
~   vpc-native-validate-active-users-cron                  kubernetes:batch/v1:CronJob               update
~   vpc-native-update-current-streak-cron                  kubernetes:batch/v1:CronJob               update

Comment on lines +598 to +602
hasProfileImage,
hasHeadline,
hasExperienceLevel,
hasWork,
hasEducation,
Copy link
Contributor

Choose a reason for hiding this comment

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

It's just a lot of extra queries on boot right?
Do we actually need anything but the percentage on actual boot? those has things all kinda only needed when you click it right? so wondering if we could optimize it a bit.

Copy link
Member Author

Choose a reason for hiding this comment

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

it's one additional query. i use this info to know where to link the user and adjust the copy.
if it slows down boot, i'll cache it

Copy link
Member Author

Choose a reason for hiding this comment

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

I checked boot latency, it doesn't seem to be impacted

@idoshamun idoshamun enabled auto-merge (squash) December 29, 2025 07:40
@idoshamun idoshamun disabled auto-merge December 29, 2025 07:41
@idoshamun idoshamun merged commit 5956454 into main Dec 29, 2025
14 checks passed
@idoshamun idoshamun deleted the profile-completion branch December 29, 2025 07:41
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.

4 participants