fix: update only relevant fields in analytics loader #4498
Merged
+2
−2
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
zawan-ila
changed the title
AfaqShuaib09
approved these changes
Nov 28, 2024
DawoudSheraz
approved these changes
Nov 28, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PROD-3899
Currently, in the analytics loader, program objects may be "cached" in memory until all course runs and courses have been processed, and then eventually be saved. The processing for all courses and courseruns may take a considerable amount of time. Hence any updates to the cached program objects during this time will be lost. This PR "fixes" the program save call to only modify the enrollment_count and recent_enrollment_count fields. We have also done the same for the courserun save, even though the chances of a lost update there are tiny, as they are saved immediately after being fetched.
A similar fix was done for courses in 4487