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

Feature/performance updates #933

Merged
merged 1 commit into from
Jan 3, 2025
Merged

Conversation

levinmr
Copy link
Contributor

@levinmr levinmr commented Jan 2, 2025

No description provided.

@levinmr levinmr force-pushed the feature/performance_updates branch 4 times, most recently from d7b7160 to 951fd7a Compare January 3, 2025 15:11
…gBoss

- Initialize the knex pool instance in the top level of publisher and consumer and pass the knex instance to the classes that make DB calls.
- Update knex initialization to use the full knexfile config instead of just the connection field.
- Add PgBossKnexAdapter class to convert PgBoss SQL statements into knex raw commands.
- Pass PgBossKnexAdapter to queue client setup. This removes the separate database for queue messages.
- Remove knex pool destroy statements from the message queue consumer and from postgres publisher.
- Keep knex pool destroy statements in message queue publisher to close the pool after each interval script.
- Update CI to remove the message queue database parameter from deploy jobs
- Remove map statement from CSV formatter to avoid making a copy of the report dataset during formatting in order to reduce consumer memory usage.
- Offset timed publisher script runs so they don't ever happen in parallel to reduce publisher memory usage.
- Disable unused hourly publisher runs.
@levinmr levinmr force-pushed the feature/performance_updates branch from 951fd7a to a415911 Compare January 3, 2025 16:46
@levinmr levinmr merged commit c752e1e into develop Jan 3, 2025
4 checks passed
@levinmr levinmr deleted the feature/performance_updates branch January 3, 2025 16:47
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