Skip to content

Remove SQL Server upgrade scripts and code - #301

Open
labkey-jeckels wants to merge 2 commits into
developfrom
fb_removeSqlServer2
Open

Remove SQL Server upgrade scripts and code#301
labkey-jeckels wants to merge 2 commits into
developfrom
fb_removeSqlServer2

Conversation

@labkey-jeckels

Copy link
Copy Markdown

Rationale

We no longer support SQLServer as a primary DB

Changes

  • Remove schema upgrade scripts
  • Remove SQLServer conditional code
  • Remove SupportedDatabases in module.properties
  • Remove uses of PostgresOnlyTest
  • Create Postgres version of covidseq schema

@labkey-jeckels
labkey-jeckels requested review from a team and bbimber August 23, 2026 16:56
@labkey-jeckels labkey-jeckels self-assigned this Aug 23, 2026
@labkey-jeckels

Copy link
Copy Markdown
Author

@bbimber the interesting thing for this repo is the covidseq module. It had no Postgres upgrade scripts. I did a trivial port (to be verified by TeamCity) to prevent schema validation failures. If this is a module worth migrating, you'll want to backport these changes to 26.3 or 26.7 so there's something to migrate into.

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