Skip to content

Comments

Migrate stored procedures from ODSE to RDB/RDB_Modern#675

Merged
mpeels merged 12 commits intomainfrom
mp/app-249/remove-sp-from-odse
Feb 24, 2026
Merged

Migrate stored procedures from ODSE to RDB/RDB_Modern#675
mpeels merged 12 commits intomainfrom
mp/app-249/remove-sp-from-odse

Conversation

@mpeels
Copy link
Contributor

@mpeels mpeels commented Feb 18, 2026

JIRA

Notes

Prevents RTR from writing to the RDB.job_flow_log when configured to use RDB_Modern.

This is done by migrating the stored procedures listed below from NBS_ODSE to the configured RDB database and removing hard coded references to RDB.job_flow_log. This migration also required updating the permissions granted for the various RTR users. While adjusting these permissions, user creation scripts were consolidated.

Stored Procedure Caller
sp_organization_event Organization Service
sp_provider_event Person Service
sp_patient_race_event sp_patient_event
sp_patient_event Person Service
sp_observation_event Observation Service
sp_investigation_event Investigation Service
sp_ldf_provider_event LDF Data Service -> sp_ldf_data_event
sp_ldf_organization_event LDF Data Service -> sp_ldf_data_event
sp_ldf_patient_event LDF Data Service -> sp_ldf_data_event
sp_ldf_observation_event LDF Data Service -> sp_ldf_data_event
sp_ldf_phc_event LDF Data Service -> sp_ldf_data_event
sp_ldf_intervention_event LDF Data Service -> sp_ldf_data_event
sp_ldf_data_event LDF Data Service
sp_public_health_case_fact_datamart_event Investigation Service
sp_notification_event Investigation Service
sp_interview_event Investigation Service
sp_user_report_permissions ??? None ???
sp_auth_user_event Person Service
sp_place_event Organization Service
sp_contact_record_event Investigation Service
sp_treatment_event Investigation Service
sp_vaccination_event Investigation Service
sp_public_health_case_fact_datamart_update Investigation Service, Organization Service, Person Service

Copy link
Contributor

@ericbuckley ericbuckley left a comment

Choose a reason for hiding this comment

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

It looks like there are still quite a few of job_flow_log package name references to "NBS_ODSE."

Copy link
Contributor

@mehansen mehansen left a comment

Choose a reason for hiding this comment

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

everything else lgtm!
I agree the organization of the user creation and permissions is easier to follow this way

Copy link
Contributor

@mehansen mehansen left a comment

Choose a reason for hiding this comment

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

thanks for walking us through this!

Copy link
Contributor

@ericbuckley ericbuckley left a comment

Choose a reason for hiding this comment

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

looks good, thanks @mpeels

Copy link
Contributor

@hawley-skylight hawley-skylight left a comment

Choose a reason for hiding this comment

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

looks good. thanks!

@mpeels mpeels merged commit e15b827 into main Feb 24, 2026
1 check passed
@mpeels mpeels deleted the mp/app-249/remove-sp-from-odse branch February 24, 2026 15:07
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.

4 participants