Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BCDA-8301: Increase CCLF import memory size to 3072MB #111

Closed
wants to merge 2 commits into from

Conversation

kyeah
Copy link
Contributor

@kyeah kyeah commented Aug 7, 2024

🎫 Ticket

https://jira.cms.gov/browse/BCDA-8301

🛠 Changes

  • Ensure CCLF import has enough memory to process files

ℹ️ Context

This is a short-term resolution for issues we are seeing with our CCLF import lambda being killed for certain large files. These files are not close to meeting the 1024MB size on their own, but they are downloaded four times within the process.

We will prioritize a fix to ensure that these files are only downloaded once during processing.

https://jira.cms.gov/browse/BCDA-8302

🧪 Validation

Increased manually and verified that the lambda could process all the files we were having issues with.
CleanShot 2024-08-07 at 16 52 46@2x

@kyeah kyeah requested a review from a team as a code owner August 7, 2024 20:52
@kyeah kyeah changed the title BCDA-8301: Increase CCLF import memory size to 2048MB BCDA-8301: Increase CCLF import memory size to 3072MB Aug 7, 2024
Copy link
Member

@gsf gsf left a comment

Choose a reason for hiding this comment

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

lgtm

@kyeah
Copy link
Contributor Author

kyeah commented Aug 14, 2024

I actually just merged CMSgov/bcda-app#985 which reduces the memory consumption, so I'll go ahead and close this one out + revert the memory in AWS Console 😄

@kyeah kyeah closed this Aug 14, 2024
@kyeah kyeah deleted the kyeah-patch-1 branch August 14, 2024 15:33
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.

2 participants