search

SQLUndercover / UndercoverToolbox

A collection of cool and useful tools, procedures and scripts for the discerning DBA
https://sqlundercover.com
MIT License
84 stars 40 forks source link

Inspector - Remove linked server support #277

Closed Adedba closed 3 years ago

Adedba commented 3 years ago

Remove linked server support , just makes it harder to maintain - if required we can always add a module that does a one way push of data from the local machine over linked server to a central server at a later date.

Adedba commented 3 years ago

Linked server references removed , next step is to remove dynamic strings which were required but now no longer required and can instead be converted to normal TSQL.

Whilst I am at it I will change all DROP PROCEDURE statements to CREATE PROCEDURE AS then ALTER PROCEDURE

Adedba commented 3 years ago

Need to put the parameter back in for the proc and default it to a value as removing the parameter broke the PowerShell auto update. The ideal fix is to remove the linked server reference in the PowerShell but this would mean that users will need to update the file in order to receive the latest updates which might be an inconvenience. Will put the parameter back to satisfy the call from the autoupdate powershell but the value will not be used within the proc.