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 improvements and add airforce, army, navy, NLM agencies -- Production #937

Merged
merged 9 commits into from
Jan 3, 2025

Conversation

levinmr
Copy link
Contributor

@levinmr levinmr commented Jan 3, 2025

No description provided.

levinmr and others added 9 commits December 19, 2024 16:16
…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.
[Feature] Add airforce, army, navy, NLM agencies
[Feature] Performance improvements and add airforce, army, navy, NLM agencies -- Staging
@levinmr levinmr merged commit 1836dab into master Jan 3, 2025
18 checks passed
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