Skip to content

Remove SQL Server upgrade scripts and code - #307

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

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

Conversation

@labkey-jeckels

@labkey-jeckels labkey-jeckels commented Aug 23, 2026

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
  • Remove Naturalize CLR management and use as there are no known uses on external schemas/DBs

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

labkey-jeckels commented Aug 23, 2026

Copy link
Copy Markdown
Author

@bbimber most of this is straightforward and mechanical, but please see my notes about the CLR. No rush to review. I restored the "include" upon review since it was part of the index name (if not actually impacting the index itself on PG)

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