Skip to content

Migrate onprc_ehr scripts to PostgreSQL#1805

Open
labkey-adam wants to merge 4 commits into
release26.3-SNAPSHOTfrom
26.3_fb_onprc_ehr_scripts
Open

Migrate onprc_ehr scripts to PostgreSQL#1805
labkey-adam wants to merge 4 commits into
release26.3-SNAPSHOTfrom
26.3_fb_onprc_ehr_scripts

Conversation

@labkey-adam

@labkey-adam labkey-adam commented Jul 19, 2026

Copy link
Copy Markdown
Contributor

Rationale

We want the onprc_ehr module to run on PostgreSQL

Changes

  • Consolidate most onprc_ehr SQL upgrade scripts to onprc_ehr-0.000-25.000.sql
  • Migrate consolidated script and the remaining incremental scripts to PostgreSQL syntax
  • Clean up bootstrap script

Tasks 📍

  • Claude Code Review
  • Manual Testing
  • Test Automation
  • Verify Fix

@labkey-adam

Copy link
Copy Markdown
Contributor Author

@labkey-martyp not sure how we want to handle procedure audit.ArchiveAuditTables() (onprc_ehr-25.002-25.003.sql). PostgreSQL can't do cross-database transactions, so Gemini migrated this procedure to use different schemas in a single database.

Note that I didn't bother migrating onprc_ehr-25.000-25.001.sql or onprc_ehr-25.001-25.002.sql since these were just earlier versions of ArchiveAuditTables() that were overwritten.

@labkey-tchad

Copy link
Copy Markdown
Member

This should remove the SqlserverOnlyTest marker interface from the ONPRC tests so that we get more TeamCity coverage.

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