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

HOTT-4957 Reduce memory usage for omitted measures #1713

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jebw
Copy link
Contributor

@jebw jebw commented Feb 8, 2024

Jira link

HOTT-4957

What?

I have added/removed/altered:

  • Refactored how we load data for omitted measures tab in the differences report

Why?

I am doing this because:

  • Previously this loaded all measures for all chapters for today plus all measures for all chapters for 1 year ago into memory in a single go
  • Now this should iterate chapters and only load all measures for 1 chapter for both today and a year ago - hopefully solving the memory usage problems

Deployment risks (optional)

  • Some, this code lacks adequate tests and I don't have the time to write more so just putting this as a draft for now

Tab in differences report

Previously this loaded _all measures_ for _all chapters_ for today plus
_all measures_ for _all chapters_ for 1 year ago into memory in a single
go

Now this should iterate chapters and only load _all measures for 1
chapter_ for both today and a year ago - hopefully solving the memory
usage problems
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
1 participant