Changing the use of GETDATE() in the entire database
I need to migrate an on-premises SQL Server 2017 database to an Azure SQL database, and I'm facing some challenges since there's quite a bit of limitations to go through.
In particular, since an Azure SQL database works only in UTC time (no time zones) and we need the local time, we have to change the use of GETDATE()
everywhere in the database, which has proven to be more work than I anticipated.
I created a user defined function to get the local time that works correctly for my time zone:
CREATE FUNCTION [dbo].[getlocaldate]()
RETURNS datetime
AS
BEGIN
DECLARE @D datetimeoffset;
SET @D = CONVERT(datetimeoffset, SYSDATETIMEOFFSET()) AT TIME ZONE 'Pacific SA Standard Time';
RETURN(CONVERT(datetime,@D));
END
The issue I'm having trouble with is to actually change GETDATE()
with this function in every view, stored procedure, computed columns, default values, other constraints, etc.
What would be the best way to implement this change?
We are in the public preview of Managed Instances. It still has the same issue with GETDATE()
, so it doesn't help with this problem. Moving to Azure is a requirement. This database is used (and will be used) always in this time zone.
sql-server azure-sql-database sql-server-2017
add a comment |
I need to migrate an on-premises SQL Server 2017 database to an Azure SQL database, and I'm facing some challenges since there's quite a bit of limitations to go through.
In particular, since an Azure SQL database works only in UTC time (no time zones) and we need the local time, we have to change the use of GETDATE()
everywhere in the database, which has proven to be more work than I anticipated.
I created a user defined function to get the local time that works correctly for my time zone:
CREATE FUNCTION [dbo].[getlocaldate]()
RETURNS datetime
AS
BEGIN
DECLARE @D datetimeoffset;
SET @D = CONVERT(datetimeoffset, SYSDATETIMEOFFSET()) AT TIME ZONE 'Pacific SA Standard Time';
RETURN(CONVERT(datetime,@D));
END
The issue I'm having trouble with is to actually change GETDATE()
with this function in every view, stored procedure, computed columns, default values, other constraints, etc.
What would be the best way to implement this change?
We are in the public preview of Managed Instances. It still has the same issue with GETDATE()
, so it doesn't help with this problem. Moving to Azure is a requirement. This database is used (and will be used) always in this time zone.
sql-server azure-sql-database sql-server-2017
5
It might sound daft, but importing the database in an SSDT project, replacing the getdate() function with a custom one and then creating the new function as per your needs might work.
– Marian
Jul 4 '18 at 17:23
What problem are you having implementing the change?
– Max Vernon
Jul 4 '18 at 18:12
Sounds like you need to figure out a query that will bring back results for all objectsGETDATE()
is referenced and then figure out ways to make those change and then just keep looking over the query list until you got them all. Perhaps you can get most of them changed with one of the method below and then do the others manually. I would think even if 50% - 80% are completed via a process then the other 50% - 20% needs to be done manually, that's better than nothing, correct? I'd start with something and start getting it done for sure.
– Pimp Juice IT
Jul 10 '18 at 3:40
add a comment |
I need to migrate an on-premises SQL Server 2017 database to an Azure SQL database, and I'm facing some challenges since there's quite a bit of limitations to go through.
In particular, since an Azure SQL database works only in UTC time (no time zones) and we need the local time, we have to change the use of GETDATE()
everywhere in the database, which has proven to be more work than I anticipated.
I created a user defined function to get the local time that works correctly for my time zone:
CREATE FUNCTION [dbo].[getlocaldate]()
RETURNS datetime
AS
BEGIN
DECLARE @D datetimeoffset;
SET @D = CONVERT(datetimeoffset, SYSDATETIMEOFFSET()) AT TIME ZONE 'Pacific SA Standard Time';
RETURN(CONVERT(datetime,@D));
END
The issue I'm having trouble with is to actually change GETDATE()
with this function in every view, stored procedure, computed columns, default values, other constraints, etc.
What would be the best way to implement this change?
We are in the public preview of Managed Instances. It still has the same issue with GETDATE()
, so it doesn't help with this problem. Moving to Azure is a requirement. This database is used (and will be used) always in this time zone.
sql-server azure-sql-database sql-server-2017
I need to migrate an on-premises SQL Server 2017 database to an Azure SQL database, and I'm facing some challenges since there's quite a bit of limitations to go through.
In particular, since an Azure SQL database works only in UTC time (no time zones) and we need the local time, we have to change the use of GETDATE()
everywhere in the database, which has proven to be more work than I anticipated.
I created a user defined function to get the local time that works correctly for my time zone:
CREATE FUNCTION [dbo].[getlocaldate]()
RETURNS datetime
AS
BEGIN
DECLARE @D datetimeoffset;
SET @D = CONVERT(datetimeoffset, SYSDATETIMEOFFSET()) AT TIME ZONE 'Pacific SA Standard Time';
RETURN(CONVERT(datetime,@D));
END
The issue I'm having trouble with is to actually change GETDATE()
with this function in every view, stored procedure, computed columns, default values, other constraints, etc.
What would be the best way to implement this change?
We are in the public preview of Managed Instances. It still has the same issue with GETDATE()
, so it doesn't help with this problem. Moving to Azure is a requirement. This database is used (and will be used) always in this time zone.
sql-server azure-sql-database sql-server-2017
sql-server azure-sql-database sql-server-2017
edited Jul 5 '18 at 14:09
Paul White♦
49.5k14261415
49.5k14261415
asked Jul 4 '18 at 16:38
LamakLamak
2,3781728
2,3781728
5
It might sound daft, but importing the database in an SSDT project, replacing the getdate() function with a custom one and then creating the new function as per your needs might work.
– Marian
Jul 4 '18 at 17:23
What problem are you having implementing the change?
– Max Vernon
Jul 4 '18 at 18:12
Sounds like you need to figure out a query that will bring back results for all objectsGETDATE()
is referenced and then figure out ways to make those change and then just keep looking over the query list until you got them all. Perhaps you can get most of them changed with one of the method below and then do the others manually. I would think even if 50% - 80% are completed via a process then the other 50% - 20% needs to be done manually, that's better than nothing, correct? I'd start with something and start getting it done for sure.
– Pimp Juice IT
Jul 10 '18 at 3:40
add a comment |
5
It might sound daft, but importing the database in an SSDT project, replacing the getdate() function with a custom one and then creating the new function as per your needs might work.
– Marian
Jul 4 '18 at 17:23
What problem are you having implementing the change?
– Max Vernon
Jul 4 '18 at 18:12
Sounds like you need to figure out a query that will bring back results for all objectsGETDATE()
is referenced and then figure out ways to make those change and then just keep looking over the query list until you got them all. Perhaps you can get most of them changed with one of the method below and then do the others manually. I would think even if 50% - 80% are completed via a process then the other 50% - 20% needs to be done manually, that's better than nothing, correct? I'd start with something and start getting it done for sure.
– Pimp Juice IT
Jul 10 '18 at 3:40
5
5
It might sound daft, but importing the database in an SSDT project, replacing the getdate() function with a custom one and then creating the new function as per your needs might work.
– Marian
Jul 4 '18 at 17:23
It might sound daft, but importing the database in an SSDT project, replacing the getdate() function with a custom one and then creating the new function as per your needs might work.
– Marian
Jul 4 '18 at 17:23
What problem are you having implementing the change?
– Max Vernon
Jul 4 '18 at 18:12
What problem are you having implementing the change?
– Max Vernon
Jul 4 '18 at 18:12
Sounds like you need to figure out a query that will bring back results for all objects
GETDATE()
is referenced and then figure out ways to make those change and then just keep looking over the query list until you got them all. Perhaps you can get most of them changed with one of the method below and then do the others manually. I would think even if 50% - 80% are completed via a process then the other 50% - 20% needs to be done manually, that's better than nothing, correct? I'd start with something and start getting it done for sure.– Pimp Juice IT
Jul 10 '18 at 3:40
Sounds like you need to figure out a query that will bring back results for all objects
GETDATE()
is referenced and then figure out ways to make those change and then just keep looking over the query list until you got them all. Perhaps you can get most of them changed with one of the method below and then do the others manually. I would think even if 50% - 80% are completed via a process then the other 50% - 20% needs to be done manually, that's better than nothing, correct? I'd start with something and start getting it done for sure.– Pimp Juice IT
Jul 10 '18 at 3:40
add a comment |
6 Answers
6
active
oldest
votes
Use the SQL Server tool to export the database objects definition to a SQL file which should include: tables, views, triggers, SPs, functions, and so on
Edit the SQL file (make a backup first) using any text editor that allows you to find the text
"GETDATE()"
and replace it with"[dbo].[getlocaldate]()"
Run the edited SQL file in Azure SQL to create your database objects...
Execute the migration of data.
Here you have a reference from azure documentation: Generating Scripts for SQL Azure
Though in practice this approach is more complicated than it sounds, it is probably the correct and best answer. I've had to do tasks similar to this many scores of times and I've tried every approach available and I haven't found anything better (or even close, really). the other approachs seem great at first, but they quickly become a nightmarish quagmire of oversights and gotchas.
– RBarryYoung
Jul 11 '18 at 18:14
add a comment |
What would be the best way to implement this change?
I would work the other way around. Convert all your timestamps in the database to UTC, and just use UTC and go with the flow. If you need a timestamp in a different tz, you can create a generated column using AT TIME ZONE
(as you did above) that renders the time stamp in that specified TZ (for the app). But, I would seriously consider just having UTC returned to the app, and writing that logic -- the display logic -- in the app.
if it was just a database thing, I may consider this, but that change affects a lot of other apps and software that would need serious refactoring. So, sadly, it isn't a choice for me
– Lamak
Jul 4 '18 at 16:51
5
What guarantee will you have that none of the "apps and software" use getdate() ? i.e. sql code embedded in the apps. If you can't guarantee that, refactoring the database to use a different function will just lead to inconsistency.
– Mister Magoo
Jul 5 '18 at 0:43
@MisterMagoo It depends on the practices at the shop, quite frankly I think this is a very minor concern, and I can't see it taking as much time to ask the question to work around the problem then to actually fix i. It would be interesting if this question wasn't Azure, because I could hack it and give you more feedback. Cloud stuff sucks though: they don't support it, so you have to change something on your side. I would prefer to go the route provided in my answer and spend the time on doing it right. Also, you have no guarantees anything will work when you move to Azure, as always tias.
– Evan Carroll
Jul 5 '18 at 0:58
@EvanCarroll, sorry I just re-read my comment and I did not express my intent well! I meant to be supportive of your answer (upvoted) and raise the point that suggestions of just changing the use of getdate() to getlocaldate() in the database would be leaving them open to inconsistencies from the app side, and moreover is just a sticking plaster on a bigger problem. 100% agree with your answer, fixing the core problem is the right approach.
– Mister Magoo
Jul 5 '18 at 6:41
@MisterMagoo I understand your concern, but in this case, I can guarantee that apps and software interact with the database only through stored procedures
– Lamak
Jul 5 '18 at 15:07
|
show 3 more comments
Rather than export, manually edit, and rerun, you could try do the job directly in the database with something like:
DECLARE C CURSOR FOR
SELECT sm.definition, so.type
FROM sys.objects so
JOIN sys.all_sql_modules sm ON sm.object_id = so.object_id
WHERE so.type IN ('P', 'V')
ORDER BY so.name
DECLARE @SQL NVARCHAR(MAX), @ojtype NVARCHAR(MAX)
OPEN C
FETCH NEXT FROM C INTO @SQL, @ojtype
WHILE @@FETCH_STATUS = 0 BEGIN
IF @objtype = 'P' SET @SQL = REPLACE(@SQL, 'CREATE PROCEDURE', 'ALTER PROCEDURE')
IF @objtype = 'V' SET @SQL = REPLACE(@SQL, 'CREATE VIEW' , 'ALTER VIEW' )
SET @SQL = REPLACE(@SQL, 'GETDATE()', '[dbo].[getlocaldate]()')
--PRINT @SQL
EXEC (@SQL)
FETCH NEXT FROM C INTO @SQL, @ojtype
END
CLOSE C
DEALLOCATE C
of course extending it to deal with functions, triggers, and so forth too.
There are a few caveats:
You may need to be a bit brighter and deal with different/extra white-space between
CREATE
andPROCEDURE
/VIEW
/<other>
. Rather than theREPLACE
for that you might prefer to instead leave theCREATE
in place and execute aDROP
first, but this risks leavingsys.depends
and friends out of kilter whereALTER
may not, also ifALTER
fails you at least have the existing object still in place where withDROP
+CREATE
you may not.If you code has any "clever" smells like modifying its own schema with ad-hoc TSQL then you'll need to make sure the search and replace for
CREATE
->ALTER
doesn't interfere with that.You will be wanting to regression test the entire application(s) after the operation, whether you use the cursor or export+edit+run methods.
I've used this method to make similar schema-wide updates in the past. It is a bit of a hack, and feels quite ugly, but sometimes it is the easiest/quickest way.
Defaults and other constraints can be modified similarly too, though those can only be dropped and recreated rather than altered. Something like:
DECLARE C CURSOR FOR
SELECT AlterDefaultSQL = 'ALTER TABLE [' +st.name+ '] DROP CONSTRAINT [' + si.name + '];'
+ CHAR(10)
+ 'ALTER TABLE [' +st.name+ '] ADD CONSTRAINT [' + si.name + '] DEFAULT '+REPLACE(si.definition, 'GETDATE()', '[dbo].[getlocaldate]()')+' FOR '+sc.name+';'
FROM sys.tables st
JOIN sys.default_constraints si ON si.parent_object_id = st.object_id
JOIN sys.columns sc ON sc.default_object_id = si.object_id
DECLARE @SQL NVARCHAR(MAX)
OPEN C
FETCH NEXT FROM C INTO @SQL
WHILE @@FETCH_STATUS = 0 BEGIN
--PRINT @SQL
EXEC (@SQL)
FETCH NEXT FROM C INTO @SQL
END
CLOSE C
DEALLOCATE C
Some more fun that you may need to contend with: if you are partitioning by time then those parts may need altering too. While partitioning on time more granularly then by day is rare you could have issues where DATETIME
s are interpreted by the partitioning function as being the previous or next day depending on timezine, leaving your partitions unaligned with your usual queries.
yeah, the caveats are what makes this hard. Also, this doesn't take into account column default values. Thanks anyway
– Lamak
Jul 5 '18 at 13:40
Column default values and other constraints can also be scanned for in thesys
schema and programmatically modified.
– David Spillett
Jul 5 '18 at 14:21
Maybe replacing with e.g.CREATE OR ALTER PROCEDURE
helps around some code-generation issues; still there may be issues as the stored definition will readCREATE PROCEDURE
(three! spaces) and this is neither matched byCREATE PROCEDURE
norCREATE OR ALTER PROCEDURE
… ._.
– TheConstructor
Jul 10 '18 at 18:11
@TheConstructor - that is what I was referring to wrt "extra whitespace". You could get around this by writing a function that scans for the firstCREATE
that is not inside a comment and replaces that. I've don't this/similar in the past but don't have the function's code handy right now to post. Or if you can guarantee none of your object definitions have comments precedingCREATE
, ignore the comments issue and just find and replace the first instance ofCREATE
.
– David Spillett
Jul 11 '18 at 10:16
I have tried this approach myself numerous times in the past and on-balance the Generate-Scripts approach was better and is almost always what I use today unless the number of objects to be changed turns out to be relatively small.
– RBarryYoung
Jul 11 '18 at 18:23
add a comment |
I really like David's answer and upvoted that for a programmatic way of doing things.
But you can try this today for a test-run in Azure via SSMS:
Right click your database --> Tasks --> Generate Scripts..
[Back Story] we had a junior DBA who upgraded all our test environments to SQL 2008 R2 while our production environments were at SQL 2008. It's a change that makes me cringe to this day. To migrate to production, from test, we had to generate scripts within SQL, using generate scripts, and in the advanced options we used the 'Type of Data to script: Schema and Data' option to generate a massive text file. We were successfully able to move our test R2 databases to our legacy SQL 2008 servers -- where a database restore to a lower version would not have worked. We used sqlcmd to input the large file--as the files were often too big for the SSMS text buffer.
What I'm saying here is that this option would probably work for you as well. You'll just need to do one additional step and search and replace getdate() with [dbo].getlocaldate in the generated text file. (I would put your function into the database before migration though).
(I never wanted to be proficient at this band-aid of a database restore, but for a while it became a defacto way of doing things. And, it worked every time.)
If you choose this route, be sure and select the Advanced button and select all the options you need (read each one) to move from the old database to the new database--like the defaults you mentioned. But give it a few test runs in Azure. I bet you'll find that this is one solution that works -- with a modicum of effort.
This is what I'd probably do. I'd have the most faith in this. Or do a generate before and after a programmatic fix to eyeball all the changes
– Paul
Jul 11 '18 at 7:48
add a comment |
Dynamically alter all proc and udf to change value
DECLARE @Text NVARCHAR(max),
@spname NVARCHAR(max),
@Type CHAR(5),
@Sql NVARCHAR(max)
DECLARE @getobject CURSOR
SET @getobject = CURSOR
FOR SELECT sc.text,
so.NAME,
so.type
FROM sys.syscomments sc
INNER JOIN sysobjects so
ON sc.id = so.id
WHERE sc.[text] LIKE '%getdate()%'
--and type in('P','FN')
OPEN @getobject
FETCH next FROM @getobject INTO @Text, @spname, @Type
WHILE @@FETCH_STATUS = 0
BEGIN
IF ( @Type = 'P'
OR @Type = 'FN' )
SET @Text = Replace(@Text, 'getdate', 'dbo.getlocaldate')
SET @Text = Replace(@Text, 'create', 'alter')
EXECUTE Sp_executesql
@Text
PRINT @Text
--,@spname,@Type
FETCH next FROM @getobject INTO @Text, @spname, @Type
END
CLOSE @getobject
DEALLOCATE @getobject
CREATE PROCEDURE [dbo].[Testproc1]
AS
SET nocount ON;
BEGIN
DECLARE @CurDate DATETIME = Getdate()
END
Notice commented sysobjects Type column condition.My script will alter only proc and UDF.
This script will alter all Default Constraint
which contain GetDate()
DECLARE @TableName VARCHAR(300),
@constraintName VARCHAR(300),
@colName VARCHAR(300),
@Sql NVARCHAR(max)
DECLARE @getobject CURSOR
SET @getobject = CURSOR
FOR SELECT ds.NAME,
sc.NAME AS colName,
so.NAME AS Tablename
--,ds.definition
FROM sys.default_constraints ds
INNER JOIN sys.columns sc
ON ds.object_id = sc.default_object_id
INNER JOIN sys.objects so
ON so.object_id = ds.parent_object_id
WHERE definition LIKE '%getdate()%'
OPEN @getobject
FETCH next FROM @getobject INTO @constraintName, @colName, @TableName
WHILE @@FETCH_STATUS = 0
BEGIN
SET @Sql = 'ALTER TABLE ' + @TableName
+ ' DROP CONSTRAINT ' + @constraintName + '; '
+ Char(13) + Char(10) + ' ' + Char(13) + Char(10) + ''
SET @Sql = @Sql + ' ALTER TABLE ' + @TableName
+ ' ADD CONSTRAINT ' + @constraintName
+ ' DEFAULT dbo.GetLocaledate() FOR '
+ @colName + ';' + Char(13) + Char(10) + ' ' + Char(13)
+ Char(10) + ''
PRINT @Sql
EXECUTE sys.Sp_executesql
@Sql
--,@spname,@Type
FETCH next FROM @getobject INTO @constraintName, @colName, @TableName
END
CLOSE @getobject
DEALLOCATE @getobject
add a comment |
I have upvoted Evan Carrolls answer, as I think this is the best solution. I have not been able to convince my colleagues that they should change a lot of C# code, so I had to use the code that David Spillett wrote. I have fixed a couple of problems with Dynamic SQL and Schemas (not all code use "dbo.") like this:
DECLARE C CURSOR LOCAL STATIC FOR
SELECT sm.definition, so.type
FROM sys.objects so
JOIN sys.all_sql_modules sm ON sm.object_id = so.object_id
WHERE so.type IN ('P', 'V')
AND CHARINDEX('getdate()', sm.definition) > 0
ORDER BY so.name
DECLARE @SQL NVARCHAR(MAX), @objtype NVARCHAR(MAX)
OPEN C
WHILE 1=1 BEGIN
FETCH NEXT FROM C INTO @SQL, @objtype
IF @@FETCH_STATUS <> 0 BREAK
IF @objtype = 'P' SET @SQL = REPLACE(@SQL, 'CREATE PROCEDURE', 'ALTER PROCEDURE')
IF @objtype = 'P' SET @SQL = REPLACE(@SQL, 'CREATE PROCEDURE', 'ALTER PROCEDURE') /* when you write "create or alter proc" */
IF @objtype = 'V' SET @SQL = REPLACE(@SQL, 'CREATE VIEW' , 'ALTER VIEW' )
IF CHARINDEX('getdate())''', @sql) > 0 BEGIN /* when dynamic SQL is used */
IF CHARINDEX('utl.getdate())''', @sql) = 0 SET @SQL = REPLACE(@SQL, 'GETDATE()', 'utl.getdate()')
end
ELSE begin
SET @SQL = REPLACE(@SQL, 'GETDATE()', 'CONVERT(DATETIME, CONVERT(datetimeoffset, SYSDATETIME()) AT TIME ZONE ''Central Europe Standard Time'')')
end
EXEC dbo.LongPrint @String = @sql
EXEC (@SQL)
END
CLOSE C
DEALLOCATE C
and the default constraints like this:
DECLARE C CURSOR LOCAL STATIC FOR
SELECT AlterDefaultSQL = 'ALTER TABLE [' +sch.name+ '].[' +st.name+ '] DROP CONSTRAINT [' + si.name + '];'
+ CHAR(10)
+ 'ALTER TABLE [' +sch.name+ '].[' +st.name+ '] ADD CONSTRAINT [' + si.name + '] DEFAULT '+REPLACE(si.definition, 'GETDATE()', 'CONVERT(DATETIME, CONVERT(datetimeoffset, SYSDATETIME()) AT TIME ZONE ''Central Europe Standard Time'')')+' FOR '+sc.name+';'
FROM sys.tables st
JOIN sys.default_constraints si ON si.parent_object_id = st.object_id
JOIN sys.columns sc ON sc.default_object_id = si.object_id
INNER JOIN sys.schemas sch ON sch.schema_id = st.schema_id
WHERE CHARINDEX('getdate()', si.definition) > 0
ORDER BY st.name, sc.name
DECLARE @SQL NVARCHAR(MAX)
OPEN C
WHILE 1=1 BEGIN
FETCH NEXT FROM C INTO @SQL
IF @@FETCH_STATUS <> 0 BREAK
EXEC dbo.LongPrint @String = @sql
EXEC (@SQL)
FETCH NEXT FROM C INTO @SQL
END
CLOSE C
DEALLOCATE C
add a comment |
Your Answer
StackExchange.ready(function() {
var channelOptions = {
tags: "".split(" "),
id: "182"
};
initTagRenderer("".split(" "), "".split(" "), channelOptions);
StackExchange.using("externalEditor", function() {
// Have to fire editor after snippets, if snippets enabled
if (StackExchange.settings.snippets.snippetsEnabled) {
StackExchange.using("snippets", function() {
createEditor();
});
}
else {
createEditor();
}
});
function createEditor() {
StackExchange.prepareEditor({
heartbeatType: 'answer',
autoActivateHeartbeat: false,
convertImagesToLinks: false,
noModals: true,
showLowRepImageUploadWarning: true,
reputationToPostImages: null,
bindNavPrevention: true,
postfix: "",
imageUploader: {
brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
allowUrls: true
},
onDemand: true,
discardSelector: ".discard-answer"
,immediatelyShowMarkdownHelp:true
});
}
});
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
StackExchange.ready(
function () {
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fdba.stackexchange.com%2fquestions%2f211352%2fchanging-the-use-of-getdate-in-the-entire-database%23new-answer', 'question_page');
}
);
Post as a guest
Required, but never shown
6 Answers
6
active
oldest
votes
6 Answers
6
active
oldest
votes
active
oldest
votes
active
oldest
votes
Use the SQL Server tool to export the database objects definition to a SQL file which should include: tables, views, triggers, SPs, functions, and so on
Edit the SQL file (make a backup first) using any text editor that allows you to find the text
"GETDATE()"
and replace it with"[dbo].[getlocaldate]()"
Run the edited SQL file in Azure SQL to create your database objects...
Execute the migration of data.
Here you have a reference from azure documentation: Generating Scripts for SQL Azure
Though in practice this approach is more complicated than it sounds, it is probably the correct and best answer. I've had to do tasks similar to this many scores of times and I've tried every approach available and I haven't found anything better (or even close, really). the other approachs seem great at first, but they quickly become a nightmarish quagmire of oversights and gotchas.
– RBarryYoung
Jul 11 '18 at 18:14
add a comment |
Use the SQL Server tool to export the database objects definition to a SQL file which should include: tables, views, triggers, SPs, functions, and so on
Edit the SQL file (make a backup first) using any text editor that allows you to find the text
"GETDATE()"
and replace it with"[dbo].[getlocaldate]()"
Run the edited SQL file in Azure SQL to create your database objects...
Execute the migration of data.
Here you have a reference from azure documentation: Generating Scripts for SQL Azure
Though in practice this approach is more complicated than it sounds, it is probably the correct and best answer. I've had to do tasks similar to this many scores of times and I've tried every approach available and I haven't found anything better (or even close, really). the other approachs seem great at first, but they quickly become a nightmarish quagmire of oversights and gotchas.
– RBarryYoung
Jul 11 '18 at 18:14
add a comment |
Use the SQL Server tool to export the database objects definition to a SQL file which should include: tables, views, triggers, SPs, functions, and so on
Edit the SQL file (make a backup first) using any text editor that allows you to find the text
"GETDATE()"
and replace it with"[dbo].[getlocaldate]()"
Run the edited SQL file in Azure SQL to create your database objects...
Execute the migration of data.
Here you have a reference from azure documentation: Generating Scripts for SQL Azure
Use the SQL Server tool to export the database objects definition to a SQL file which should include: tables, views, triggers, SPs, functions, and so on
Edit the SQL file (make a backup first) using any text editor that allows you to find the text
"GETDATE()"
and replace it with"[dbo].[getlocaldate]()"
Run the edited SQL file in Azure SQL to create your database objects...
Execute the migration of data.
Here you have a reference from azure documentation: Generating Scripts for SQL Azure
edited Jul 5 '18 at 13:38
Erik Darling
21.2k1264104
21.2k1264104
answered Jul 4 '18 at 19:17
AMGAMG
1,1529
1,1529
Though in practice this approach is more complicated than it sounds, it is probably the correct and best answer. I've had to do tasks similar to this many scores of times and I've tried every approach available and I haven't found anything better (or even close, really). the other approachs seem great at first, but they quickly become a nightmarish quagmire of oversights and gotchas.
– RBarryYoung
Jul 11 '18 at 18:14
add a comment |
Though in practice this approach is more complicated than it sounds, it is probably the correct and best answer. I've had to do tasks similar to this many scores of times and I've tried every approach available and I haven't found anything better (or even close, really). the other approachs seem great at first, but they quickly become a nightmarish quagmire of oversights and gotchas.
– RBarryYoung
Jul 11 '18 at 18:14
Though in practice this approach is more complicated than it sounds, it is probably the correct and best answer. I've had to do tasks similar to this many scores of times and I've tried every approach available and I haven't found anything better (or even close, really). the other approachs seem great at first, but they quickly become a nightmarish quagmire of oversights and gotchas.
– RBarryYoung
Jul 11 '18 at 18:14
Though in practice this approach is more complicated than it sounds, it is probably the correct and best answer. I've had to do tasks similar to this many scores of times and I've tried every approach available and I haven't found anything better (or even close, really). the other approachs seem great at first, but they quickly become a nightmarish quagmire of oversights and gotchas.
– RBarryYoung
Jul 11 '18 at 18:14
add a comment |
What would be the best way to implement this change?
I would work the other way around. Convert all your timestamps in the database to UTC, and just use UTC and go with the flow. If you need a timestamp in a different tz, you can create a generated column using AT TIME ZONE
(as you did above) that renders the time stamp in that specified TZ (for the app). But, I would seriously consider just having UTC returned to the app, and writing that logic -- the display logic -- in the app.
if it was just a database thing, I may consider this, but that change affects a lot of other apps and software that would need serious refactoring. So, sadly, it isn't a choice for me
– Lamak
Jul 4 '18 at 16:51
5
What guarantee will you have that none of the "apps and software" use getdate() ? i.e. sql code embedded in the apps. If you can't guarantee that, refactoring the database to use a different function will just lead to inconsistency.
– Mister Magoo
Jul 5 '18 at 0:43
@MisterMagoo It depends on the practices at the shop, quite frankly I think this is a very minor concern, and I can't see it taking as much time to ask the question to work around the problem then to actually fix i. It would be interesting if this question wasn't Azure, because I could hack it and give you more feedback. Cloud stuff sucks though: they don't support it, so you have to change something on your side. I would prefer to go the route provided in my answer and spend the time on doing it right. Also, you have no guarantees anything will work when you move to Azure, as always tias.
– Evan Carroll
Jul 5 '18 at 0:58
@EvanCarroll, sorry I just re-read my comment and I did not express my intent well! I meant to be supportive of your answer (upvoted) and raise the point that suggestions of just changing the use of getdate() to getlocaldate() in the database would be leaving them open to inconsistencies from the app side, and moreover is just a sticking plaster on a bigger problem. 100% agree with your answer, fixing the core problem is the right approach.
– Mister Magoo
Jul 5 '18 at 6:41
@MisterMagoo I understand your concern, but in this case, I can guarantee that apps and software interact with the database only through stored procedures
– Lamak
Jul 5 '18 at 15:07
|
show 3 more comments
What would be the best way to implement this change?
I would work the other way around. Convert all your timestamps in the database to UTC, and just use UTC and go with the flow. If you need a timestamp in a different tz, you can create a generated column using AT TIME ZONE
(as you did above) that renders the time stamp in that specified TZ (for the app). But, I would seriously consider just having UTC returned to the app, and writing that logic -- the display logic -- in the app.
if it was just a database thing, I may consider this, but that change affects a lot of other apps and software that would need serious refactoring. So, sadly, it isn't a choice for me
– Lamak
Jul 4 '18 at 16:51
5
What guarantee will you have that none of the "apps and software" use getdate() ? i.e. sql code embedded in the apps. If you can't guarantee that, refactoring the database to use a different function will just lead to inconsistency.
– Mister Magoo
Jul 5 '18 at 0:43
@MisterMagoo It depends on the practices at the shop, quite frankly I think this is a very minor concern, and I can't see it taking as much time to ask the question to work around the problem then to actually fix i. It would be interesting if this question wasn't Azure, because I could hack it and give you more feedback. Cloud stuff sucks though: they don't support it, so you have to change something on your side. I would prefer to go the route provided in my answer and spend the time on doing it right. Also, you have no guarantees anything will work when you move to Azure, as always tias.
– Evan Carroll
Jul 5 '18 at 0:58
@EvanCarroll, sorry I just re-read my comment and I did not express my intent well! I meant to be supportive of your answer (upvoted) and raise the point that suggestions of just changing the use of getdate() to getlocaldate() in the database would be leaving them open to inconsistencies from the app side, and moreover is just a sticking plaster on a bigger problem. 100% agree with your answer, fixing the core problem is the right approach.
– Mister Magoo
Jul 5 '18 at 6:41
@MisterMagoo I understand your concern, but in this case, I can guarantee that apps and software interact with the database only through stored procedures
– Lamak
Jul 5 '18 at 15:07
|
show 3 more comments
What would be the best way to implement this change?
I would work the other way around. Convert all your timestamps in the database to UTC, and just use UTC and go with the flow. If you need a timestamp in a different tz, you can create a generated column using AT TIME ZONE
(as you did above) that renders the time stamp in that specified TZ (for the app). But, I would seriously consider just having UTC returned to the app, and writing that logic -- the display logic -- in the app.
What would be the best way to implement this change?
I would work the other way around. Convert all your timestamps in the database to UTC, and just use UTC and go with the flow. If you need a timestamp in a different tz, you can create a generated column using AT TIME ZONE
(as you did above) that renders the time stamp in that specified TZ (for the app). But, I would seriously consider just having UTC returned to the app, and writing that logic -- the display logic -- in the app.
edited Jul 4 '18 at 16:51
answered Jul 4 '18 at 16:49
Evan CarrollEvan Carroll
31.7k967215
31.7k967215
if it was just a database thing, I may consider this, but that change affects a lot of other apps and software that would need serious refactoring. So, sadly, it isn't a choice for me
– Lamak
Jul 4 '18 at 16:51
5
What guarantee will you have that none of the "apps and software" use getdate() ? i.e. sql code embedded in the apps. If you can't guarantee that, refactoring the database to use a different function will just lead to inconsistency.
– Mister Magoo
Jul 5 '18 at 0:43
@MisterMagoo It depends on the practices at the shop, quite frankly I think this is a very minor concern, and I can't see it taking as much time to ask the question to work around the problem then to actually fix i. It would be interesting if this question wasn't Azure, because I could hack it and give you more feedback. Cloud stuff sucks though: they don't support it, so you have to change something on your side. I would prefer to go the route provided in my answer and spend the time on doing it right. Also, you have no guarantees anything will work when you move to Azure, as always tias.
– Evan Carroll
Jul 5 '18 at 0:58
@EvanCarroll, sorry I just re-read my comment and I did not express my intent well! I meant to be supportive of your answer (upvoted) and raise the point that suggestions of just changing the use of getdate() to getlocaldate() in the database would be leaving them open to inconsistencies from the app side, and moreover is just a sticking plaster on a bigger problem. 100% agree with your answer, fixing the core problem is the right approach.
– Mister Magoo
Jul 5 '18 at 6:41
@MisterMagoo I understand your concern, but in this case, I can guarantee that apps and software interact with the database only through stored procedures
– Lamak
Jul 5 '18 at 15:07
|
show 3 more comments
if it was just a database thing, I may consider this, but that change affects a lot of other apps and software that would need serious refactoring. So, sadly, it isn't a choice for me
– Lamak
Jul 4 '18 at 16:51
5
What guarantee will you have that none of the "apps and software" use getdate() ? i.e. sql code embedded in the apps. If you can't guarantee that, refactoring the database to use a different function will just lead to inconsistency.
– Mister Magoo
Jul 5 '18 at 0:43
@MisterMagoo It depends on the practices at the shop, quite frankly I think this is a very minor concern, and I can't see it taking as much time to ask the question to work around the problem then to actually fix i. It would be interesting if this question wasn't Azure, because I could hack it and give you more feedback. Cloud stuff sucks though: they don't support it, so you have to change something on your side. I would prefer to go the route provided in my answer and spend the time on doing it right. Also, you have no guarantees anything will work when you move to Azure, as always tias.
– Evan Carroll
Jul 5 '18 at 0:58
@EvanCarroll, sorry I just re-read my comment and I did not express my intent well! I meant to be supportive of your answer (upvoted) and raise the point that suggestions of just changing the use of getdate() to getlocaldate() in the database would be leaving them open to inconsistencies from the app side, and moreover is just a sticking plaster on a bigger problem. 100% agree with your answer, fixing the core problem is the right approach.
– Mister Magoo
Jul 5 '18 at 6:41
@MisterMagoo I understand your concern, but in this case, I can guarantee that apps and software interact with the database only through stored procedures
– Lamak
Jul 5 '18 at 15:07
if it was just a database thing, I may consider this, but that change affects a lot of other apps and software that would need serious refactoring. So, sadly, it isn't a choice for me
– Lamak
Jul 4 '18 at 16:51
if it was just a database thing, I may consider this, but that change affects a lot of other apps and software that would need serious refactoring. So, sadly, it isn't a choice for me
– Lamak
Jul 4 '18 at 16:51
5
5
What guarantee will you have that none of the "apps and software" use getdate() ? i.e. sql code embedded in the apps. If you can't guarantee that, refactoring the database to use a different function will just lead to inconsistency.
– Mister Magoo
Jul 5 '18 at 0:43
What guarantee will you have that none of the "apps and software" use getdate() ? i.e. sql code embedded in the apps. If you can't guarantee that, refactoring the database to use a different function will just lead to inconsistency.
– Mister Magoo
Jul 5 '18 at 0:43
@MisterMagoo It depends on the practices at the shop, quite frankly I think this is a very minor concern, and I can't see it taking as much time to ask the question to work around the problem then to actually fix i. It would be interesting if this question wasn't Azure, because I could hack it and give you more feedback. Cloud stuff sucks though: they don't support it, so you have to change something on your side. I would prefer to go the route provided in my answer and spend the time on doing it right. Also, you have no guarantees anything will work when you move to Azure, as always tias.
– Evan Carroll
Jul 5 '18 at 0:58
@MisterMagoo It depends on the practices at the shop, quite frankly I think this is a very minor concern, and I can't see it taking as much time to ask the question to work around the problem then to actually fix i. It would be interesting if this question wasn't Azure, because I could hack it and give you more feedback. Cloud stuff sucks though: they don't support it, so you have to change something on your side. I would prefer to go the route provided in my answer and spend the time on doing it right. Also, you have no guarantees anything will work when you move to Azure, as always tias.
– Evan Carroll
Jul 5 '18 at 0:58
@EvanCarroll, sorry I just re-read my comment and I did not express my intent well! I meant to be supportive of your answer (upvoted) and raise the point that suggestions of just changing the use of getdate() to getlocaldate() in the database would be leaving them open to inconsistencies from the app side, and moreover is just a sticking plaster on a bigger problem. 100% agree with your answer, fixing the core problem is the right approach.
– Mister Magoo
Jul 5 '18 at 6:41
@EvanCarroll, sorry I just re-read my comment and I did not express my intent well! I meant to be supportive of your answer (upvoted) and raise the point that suggestions of just changing the use of getdate() to getlocaldate() in the database would be leaving them open to inconsistencies from the app side, and moreover is just a sticking plaster on a bigger problem. 100% agree with your answer, fixing the core problem is the right approach.
– Mister Magoo
Jul 5 '18 at 6:41
@MisterMagoo I understand your concern, but in this case, I can guarantee that apps and software interact with the database only through stored procedures
– Lamak
Jul 5 '18 at 15:07
@MisterMagoo I understand your concern, but in this case, I can guarantee that apps and software interact with the database only through stored procedures
– Lamak
Jul 5 '18 at 15:07
|
show 3 more comments
Rather than export, manually edit, and rerun, you could try do the job directly in the database with something like:
DECLARE C CURSOR FOR
SELECT sm.definition, so.type
FROM sys.objects so
JOIN sys.all_sql_modules sm ON sm.object_id = so.object_id
WHERE so.type IN ('P', 'V')
ORDER BY so.name
DECLARE @SQL NVARCHAR(MAX), @ojtype NVARCHAR(MAX)
OPEN C
FETCH NEXT FROM C INTO @SQL, @ojtype
WHILE @@FETCH_STATUS = 0 BEGIN
IF @objtype = 'P' SET @SQL = REPLACE(@SQL, 'CREATE PROCEDURE', 'ALTER PROCEDURE')
IF @objtype = 'V' SET @SQL = REPLACE(@SQL, 'CREATE VIEW' , 'ALTER VIEW' )
SET @SQL = REPLACE(@SQL, 'GETDATE()', '[dbo].[getlocaldate]()')
--PRINT @SQL
EXEC (@SQL)
FETCH NEXT FROM C INTO @SQL, @ojtype
END
CLOSE C
DEALLOCATE C
of course extending it to deal with functions, triggers, and so forth too.
There are a few caveats:
You may need to be a bit brighter and deal with different/extra white-space between
CREATE
andPROCEDURE
/VIEW
/<other>
. Rather than theREPLACE
for that you might prefer to instead leave theCREATE
in place and execute aDROP
first, but this risks leavingsys.depends
and friends out of kilter whereALTER
may not, also ifALTER
fails you at least have the existing object still in place where withDROP
+CREATE
you may not.If you code has any "clever" smells like modifying its own schema with ad-hoc TSQL then you'll need to make sure the search and replace for
CREATE
->ALTER
doesn't interfere with that.You will be wanting to regression test the entire application(s) after the operation, whether you use the cursor or export+edit+run methods.
I've used this method to make similar schema-wide updates in the past. It is a bit of a hack, and feels quite ugly, but sometimes it is the easiest/quickest way.
Defaults and other constraints can be modified similarly too, though those can only be dropped and recreated rather than altered. Something like:
DECLARE C CURSOR FOR
SELECT AlterDefaultSQL = 'ALTER TABLE [' +st.name+ '] DROP CONSTRAINT [' + si.name + '];'
+ CHAR(10)
+ 'ALTER TABLE [' +st.name+ '] ADD CONSTRAINT [' + si.name + '] DEFAULT '+REPLACE(si.definition, 'GETDATE()', '[dbo].[getlocaldate]()')+' FOR '+sc.name+';'
FROM sys.tables st
JOIN sys.default_constraints si ON si.parent_object_id = st.object_id
JOIN sys.columns sc ON sc.default_object_id = si.object_id
DECLARE @SQL NVARCHAR(MAX)
OPEN C
FETCH NEXT FROM C INTO @SQL
WHILE @@FETCH_STATUS = 0 BEGIN
--PRINT @SQL
EXEC (@SQL)
FETCH NEXT FROM C INTO @SQL
END
CLOSE C
DEALLOCATE C
Some more fun that you may need to contend with: if you are partitioning by time then those parts may need altering too. While partitioning on time more granularly then by day is rare you could have issues where DATETIME
s are interpreted by the partitioning function as being the previous or next day depending on timezine, leaving your partitions unaligned with your usual queries.
yeah, the caveats are what makes this hard. Also, this doesn't take into account column default values. Thanks anyway
– Lamak
Jul 5 '18 at 13:40
Column default values and other constraints can also be scanned for in thesys
schema and programmatically modified.
– David Spillett
Jul 5 '18 at 14:21
Maybe replacing with e.g.CREATE OR ALTER PROCEDURE
helps around some code-generation issues; still there may be issues as the stored definition will readCREATE PROCEDURE
(three! spaces) and this is neither matched byCREATE PROCEDURE
norCREATE OR ALTER PROCEDURE
… ._.
– TheConstructor
Jul 10 '18 at 18:11
@TheConstructor - that is what I was referring to wrt "extra whitespace". You could get around this by writing a function that scans for the firstCREATE
that is not inside a comment and replaces that. I've don't this/similar in the past but don't have the function's code handy right now to post. Or if you can guarantee none of your object definitions have comments precedingCREATE
, ignore the comments issue and just find and replace the first instance ofCREATE
.
– David Spillett
Jul 11 '18 at 10:16
I have tried this approach myself numerous times in the past and on-balance the Generate-Scripts approach was better and is almost always what I use today unless the number of objects to be changed turns out to be relatively small.
– RBarryYoung
Jul 11 '18 at 18:23
add a comment |
Rather than export, manually edit, and rerun, you could try do the job directly in the database with something like:
DECLARE C CURSOR FOR
SELECT sm.definition, so.type
FROM sys.objects so
JOIN sys.all_sql_modules sm ON sm.object_id = so.object_id
WHERE so.type IN ('P', 'V')
ORDER BY so.name
DECLARE @SQL NVARCHAR(MAX), @ojtype NVARCHAR(MAX)
OPEN C
FETCH NEXT FROM C INTO @SQL, @ojtype
WHILE @@FETCH_STATUS = 0 BEGIN
IF @objtype = 'P' SET @SQL = REPLACE(@SQL, 'CREATE PROCEDURE', 'ALTER PROCEDURE')
IF @objtype = 'V' SET @SQL = REPLACE(@SQL, 'CREATE VIEW' , 'ALTER VIEW' )
SET @SQL = REPLACE(@SQL, 'GETDATE()', '[dbo].[getlocaldate]()')
--PRINT @SQL
EXEC (@SQL)
FETCH NEXT FROM C INTO @SQL, @ojtype
END
CLOSE C
DEALLOCATE C
of course extending it to deal with functions, triggers, and so forth too.
There are a few caveats:
You may need to be a bit brighter and deal with different/extra white-space between
CREATE
andPROCEDURE
/VIEW
/<other>
. Rather than theREPLACE
for that you might prefer to instead leave theCREATE
in place and execute aDROP
first, but this risks leavingsys.depends
and friends out of kilter whereALTER
may not, also ifALTER
fails you at least have the existing object still in place where withDROP
+CREATE
you may not.If you code has any "clever" smells like modifying its own schema with ad-hoc TSQL then you'll need to make sure the search and replace for
CREATE
->ALTER
doesn't interfere with that.You will be wanting to regression test the entire application(s) after the operation, whether you use the cursor or export+edit+run methods.
I've used this method to make similar schema-wide updates in the past. It is a bit of a hack, and feels quite ugly, but sometimes it is the easiest/quickest way.
Defaults and other constraints can be modified similarly too, though those can only be dropped and recreated rather than altered. Something like:
DECLARE C CURSOR FOR
SELECT AlterDefaultSQL = 'ALTER TABLE [' +st.name+ '] DROP CONSTRAINT [' + si.name + '];'
+ CHAR(10)
+ 'ALTER TABLE [' +st.name+ '] ADD CONSTRAINT [' + si.name + '] DEFAULT '+REPLACE(si.definition, 'GETDATE()', '[dbo].[getlocaldate]()')+' FOR '+sc.name+';'
FROM sys.tables st
JOIN sys.default_constraints si ON si.parent_object_id = st.object_id
JOIN sys.columns sc ON sc.default_object_id = si.object_id
DECLARE @SQL NVARCHAR(MAX)
OPEN C
FETCH NEXT FROM C INTO @SQL
WHILE @@FETCH_STATUS = 0 BEGIN
--PRINT @SQL
EXEC (@SQL)
FETCH NEXT FROM C INTO @SQL
END
CLOSE C
DEALLOCATE C
Some more fun that you may need to contend with: if you are partitioning by time then those parts may need altering too. While partitioning on time more granularly then by day is rare you could have issues where DATETIME
s are interpreted by the partitioning function as being the previous or next day depending on timezine, leaving your partitions unaligned with your usual queries.
yeah, the caveats are what makes this hard. Also, this doesn't take into account column default values. Thanks anyway
– Lamak
Jul 5 '18 at 13:40
Column default values and other constraints can also be scanned for in thesys
schema and programmatically modified.
– David Spillett
Jul 5 '18 at 14:21
Maybe replacing with e.g.CREATE OR ALTER PROCEDURE
helps around some code-generation issues; still there may be issues as the stored definition will readCREATE PROCEDURE
(three! spaces) and this is neither matched byCREATE PROCEDURE
norCREATE OR ALTER PROCEDURE
… ._.
– TheConstructor
Jul 10 '18 at 18:11
@TheConstructor - that is what I was referring to wrt "extra whitespace". You could get around this by writing a function that scans for the firstCREATE
that is not inside a comment and replaces that. I've don't this/similar in the past but don't have the function's code handy right now to post. Or if you can guarantee none of your object definitions have comments precedingCREATE
, ignore the comments issue and just find and replace the first instance ofCREATE
.
– David Spillett
Jul 11 '18 at 10:16
I have tried this approach myself numerous times in the past and on-balance the Generate-Scripts approach was better and is almost always what I use today unless the number of objects to be changed turns out to be relatively small.
– RBarryYoung
Jul 11 '18 at 18:23
add a comment |
Rather than export, manually edit, and rerun, you could try do the job directly in the database with something like:
DECLARE C CURSOR FOR
SELECT sm.definition, so.type
FROM sys.objects so
JOIN sys.all_sql_modules sm ON sm.object_id = so.object_id
WHERE so.type IN ('P', 'V')
ORDER BY so.name
DECLARE @SQL NVARCHAR(MAX), @ojtype NVARCHAR(MAX)
OPEN C
FETCH NEXT FROM C INTO @SQL, @ojtype
WHILE @@FETCH_STATUS = 0 BEGIN
IF @objtype = 'P' SET @SQL = REPLACE(@SQL, 'CREATE PROCEDURE', 'ALTER PROCEDURE')
IF @objtype = 'V' SET @SQL = REPLACE(@SQL, 'CREATE VIEW' , 'ALTER VIEW' )
SET @SQL = REPLACE(@SQL, 'GETDATE()', '[dbo].[getlocaldate]()')
--PRINT @SQL
EXEC (@SQL)
FETCH NEXT FROM C INTO @SQL, @ojtype
END
CLOSE C
DEALLOCATE C
of course extending it to deal with functions, triggers, and so forth too.
There are a few caveats:
You may need to be a bit brighter and deal with different/extra white-space between
CREATE
andPROCEDURE
/VIEW
/<other>
. Rather than theREPLACE
for that you might prefer to instead leave theCREATE
in place and execute aDROP
first, but this risks leavingsys.depends
and friends out of kilter whereALTER
may not, also ifALTER
fails you at least have the existing object still in place where withDROP
+CREATE
you may not.If you code has any "clever" smells like modifying its own schema with ad-hoc TSQL then you'll need to make sure the search and replace for
CREATE
->ALTER
doesn't interfere with that.You will be wanting to regression test the entire application(s) after the operation, whether you use the cursor or export+edit+run methods.
I've used this method to make similar schema-wide updates in the past. It is a bit of a hack, and feels quite ugly, but sometimes it is the easiest/quickest way.
Defaults and other constraints can be modified similarly too, though those can only be dropped and recreated rather than altered. Something like:
DECLARE C CURSOR FOR
SELECT AlterDefaultSQL = 'ALTER TABLE [' +st.name+ '] DROP CONSTRAINT [' + si.name + '];'
+ CHAR(10)
+ 'ALTER TABLE [' +st.name+ '] ADD CONSTRAINT [' + si.name + '] DEFAULT '+REPLACE(si.definition, 'GETDATE()', '[dbo].[getlocaldate]()')+' FOR '+sc.name+';'
FROM sys.tables st
JOIN sys.default_constraints si ON si.parent_object_id = st.object_id
JOIN sys.columns sc ON sc.default_object_id = si.object_id
DECLARE @SQL NVARCHAR(MAX)
OPEN C
FETCH NEXT FROM C INTO @SQL
WHILE @@FETCH_STATUS = 0 BEGIN
--PRINT @SQL
EXEC (@SQL)
FETCH NEXT FROM C INTO @SQL
END
CLOSE C
DEALLOCATE C
Some more fun that you may need to contend with: if you are partitioning by time then those parts may need altering too. While partitioning on time more granularly then by day is rare you could have issues where DATETIME
s are interpreted by the partitioning function as being the previous or next day depending on timezine, leaving your partitions unaligned with your usual queries.
Rather than export, manually edit, and rerun, you could try do the job directly in the database with something like:
DECLARE C CURSOR FOR
SELECT sm.definition, so.type
FROM sys.objects so
JOIN sys.all_sql_modules sm ON sm.object_id = so.object_id
WHERE so.type IN ('P', 'V')
ORDER BY so.name
DECLARE @SQL NVARCHAR(MAX), @ojtype NVARCHAR(MAX)
OPEN C
FETCH NEXT FROM C INTO @SQL, @ojtype
WHILE @@FETCH_STATUS = 0 BEGIN
IF @objtype = 'P' SET @SQL = REPLACE(@SQL, 'CREATE PROCEDURE', 'ALTER PROCEDURE')
IF @objtype = 'V' SET @SQL = REPLACE(@SQL, 'CREATE VIEW' , 'ALTER VIEW' )
SET @SQL = REPLACE(@SQL, 'GETDATE()', '[dbo].[getlocaldate]()')
--PRINT @SQL
EXEC (@SQL)
FETCH NEXT FROM C INTO @SQL, @ojtype
END
CLOSE C
DEALLOCATE C
of course extending it to deal with functions, triggers, and so forth too.
There are a few caveats:
You may need to be a bit brighter and deal with different/extra white-space between
CREATE
andPROCEDURE
/VIEW
/<other>
. Rather than theREPLACE
for that you might prefer to instead leave theCREATE
in place and execute aDROP
first, but this risks leavingsys.depends
and friends out of kilter whereALTER
may not, also ifALTER
fails you at least have the existing object still in place where withDROP
+CREATE
you may not.If you code has any "clever" smells like modifying its own schema with ad-hoc TSQL then you'll need to make sure the search and replace for
CREATE
->ALTER
doesn't interfere with that.You will be wanting to regression test the entire application(s) after the operation, whether you use the cursor or export+edit+run methods.
I've used this method to make similar schema-wide updates in the past. It is a bit of a hack, and feels quite ugly, but sometimes it is the easiest/quickest way.
Defaults and other constraints can be modified similarly too, though those can only be dropped and recreated rather than altered. Something like:
DECLARE C CURSOR FOR
SELECT AlterDefaultSQL = 'ALTER TABLE [' +st.name+ '] DROP CONSTRAINT [' + si.name + '];'
+ CHAR(10)
+ 'ALTER TABLE [' +st.name+ '] ADD CONSTRAINT [' + si.name + '] DEFAULT '+REPLACE(si.definition, 'GETDATE()', '[dbo].[getlocaldate]()')+' FOR '+sc.name+';'
FROM sys.tables st
JOIN sys.default_constraints si ON si.parent_object_id = st.object_id
JOIN sys.columns sc ON sc.default_object_id = si.object_id
DECLARE @SQL NVARCHAR(MAX)
OPEN C
FETCH NEXT FROM C INTO @SQL
WHILE @@FETCH_STATUS = 0 BEGIN
--PRINT @SQL
EXEC (@SQL)
FETCH NEXT FROM C INTO @SQL
END
CLOSE C
DEALLOCATE C
Some more fun that you may need to contend with: if you are partitioning by time then those parts may need altering too. While partitioning on time more granularly then by day is rare you could have issues where DATETIME
s are interpreted by the partitioning function as being the previous or next day depending on timezine, leaving your partitions unaligned with your usual queries.
edited Jul 10 '18 at 11:26
answered Jul 5 '18 at 13:26
David SpillettDavid Spillett
22.2k23267
22.2k23267
yeah, the caveats are what makes this hard. Also, this doesn't take into account column default values. Thanks anyway
– Lamak
Jul 5 '18 at 13:40
Column default values and other constraints can also be scanned for in thesys
schema and programmatically modified.
– David Spillett
Jul 5 '18 at 14:21
Maybe replacing with e.g.CREATE OR ALTER PROCEDURE
helps around some code-generation issues; still there may be issues as the stored definition will readCREATE PROCEDURE
(three! spaces) and this is neither matched byCREATE PROCEDURE
norCREATE OR ALTER PROCEDURE
… ._.
– TheConstructor
Jul 10 '18 at 18:11
@TheConstructor - that is what I was referring to wrt "extra whitespace". You could get around this by writing a function that scans for the firstCREATE
that is not inside a comment and replaces that. I've don't this/similar in the past but don't have the function's code handy right now to post. Or if you can guarantee none of your object definitions have comments precedingCREATE
, ignore the comments issue and just find and replace the first instance ofCREATE
.
– David Spillett
Jul 11 '18 at 10:16
I have tried this approach myself numerous times in the past and on-balance the Generate-Scripts approach was better and is almost always what I use today unless the number of objects to be changed turns out to be relatively small.
– RBarryYoung
Jul 11 '18 at 18:23
add a comment |
yeah, the caveats are what makes this hard. Also, this doesn't take into account column default values. Thanks anyway
– Lamak
Jul 5 '18 at 13:40
Column default values and other constraints can also be scanned for in thesys
schema and programmatically modified.
– David Spillett
Jul 5 '18 at 14:21
Maybe replacing with e.g.CREATE OR ALTER PROCEDURE
helps around some code-generation issues; still there may be issues as the stored definition will readCREATE PROCEDURE
(three! spaces) and this is neither matched byCREATE PROCEDURE
norCREATE OR ALTER PROCEDURE
… ._.
– TheConstructor
Jul 10 '18 at 18:11
@TheConstructor - that is what I was referring to wrt "extra whitespace". You could get around this by writing a function that scans for the firstCREATE
that is not inside a comment and replaces that. I've don't this/similar in the past but don't have the function's code handy right now to post. Or if you can guarantee none of your object definitions have comments precedingCREATE
, ignore the comments issue and just find and replace the first instance ofCREATE
.
– David Spillett
Jul 11 '18 at 10:16
I have tried this approach myself numerous times in the past and on-balance the Generate-Scripts approach was better and is almost always what I use today unless the number of objects to be changed turns out to be relatively small.
– RBarryYoung
Jul 11 '18 at 18:23
yeah, the caveats are what makes this hard. Also, this doesn't take into account column default values. Thanks anyway
– Lamak
Jul 5 '18 at 13:40
yeah, the caveats are what makes this hard. Also, this doesn't take into account column default values. Thanks anyway
– Lamak
Jul 5 '18 at 13:40
Column default values and other constraints can also be scanned for in the
sys
schema and programmatically modified.– David Spillett
Jul 5 '18 at 14:21
Column default values and other constraints can also be scanned for in the
sys
schema and programmatically modified.– David Spillett
Jul 5 '18 at 14:21
Maybe replacing with e.g.
CREATE OR ALTER PROCEDURE
helps around some code-generation issues; still there may be issues as the stored definition will read CREATE PROCEDURE
(three! spaces) and this is neither matched by CREATE PROCEDURE
nor CREATE OR ALTER PROCEDURE
… ._.– TheConstructor
Jul 10 '18 at 18:11
Maybe replacing with e.g.
CREATE OR ALTER PROCEDURE
helps around some code-generation issues; still there may be issues as the stored definition will read CREATE PROCEDURE
(three! spaces) and this is neither matched by CREATE PROCEDURE
nor CREATE OR ALTER PROCEDURE
… ._.– TheConstructor
Jul 10 '18 at 18:11
@TheConstructor - that is what I was referring to wrt "extra whitespace". You could get around this by writing a function that scans for the first
CREATE
that is not inside a comment and replaces that. I've don't this/similar in the past but don't have the function's code handy right now to post. Or if you can guarantee none of your object definitions have comments preceding CREATE
, ignore the comments issue and just find and replace the first instance of CREATE
.– David Spillett
Jul 11 '18 at 10:16
@TheConstructor - that is what I was referring to wrt "extra whitespace". You could get around this by writing a function that scans for the first
CREATE
that is not inside a comment and replaces that. I've don't this/similar in the past but don't have the function's code handy right now to post. Or if you can guarantee none of your object definitions have comments preceding CREATE
, ignore the comments issue and just find and replace the first instance of CREATE
.– David Spillett
Jul 11 '18 at 10:16
I have tried this approach myself numerous times in the past and on-balance the Generate-Scripts approach was better and is almost always what I use today unless the number of objects to be changed turns out to be relatively small.
– RBarryYoung
Jul 11 '18 at 18:23
I have tried this approach myself numerous times in the past and on-balance the Generate-Scripts approach was better and is almost always what I use today unless the number of objects to be changed turns out to be relatively small.
– RBarryYoung
Jul 11 '18 at 18:23
add a comment |
I really like David's answer and upvoted that for a programmatic way of doing things.
But you can try this today for a test-run in Azure via SSMS:
Right click your database --> Tasks --> Generate Scripts..
[Back Story] we had a junior DBA who upgraded all our test environments to SQL 2008 R2 while our production environments were at SQL 2008. It's a change that makes me cringe to this day. To migrate to production, from test, we had to generate scripts within SQL, using generate scripts, and in the advanced options we used the 'Type of Data to script: Schema and Data' option to generate a massive text file. We were successfully able to move our test R2 databases to our legacy SQL 2008 servers -- where a database restore to a lower version would not have worked. We used sqlcmd to input the large file--as the files were often too big for the SSMS text buffer.
What I'm saying here is that this option would probably work for you as well. You'll just need to do one additional step and search and replace getdate() with [dbo].getlocaldate in the generated text file. (I would put your function into the database before migration though).
(I never wanted to be proficient at this band-aid of a database restore, but for a while it became a defacto way of doing things. And, it worked every time.)
If you choose this route, be sure and select the Advanced button and select all the options you need (read each one) to move from the old database to the new database--like the defaults you mentioned. But give it a few test runs in Azure. I bet you'll find that this is one solution that works -- with a modicum of effort.
This is what I'd probably do. I'd have the most faith in this. Or do a generate before and after a programmatic fix to eyeball all the changes
– Paul
Jul 11 '18 at 7:48
add a comment |
I really like David's answer and upvoted that for a programmatic way of doing things.
But you can try this today for a test-run in Azure via SSMS:
Right click your database --> Tasks --> Generate Scripts..
[Back Story] we had a junior DBA who upgraded all our test environments to SQL 2008 R2 while our production environments were at SQL 2008. It's a change that makes me cringe to this day. To migrate to production, from test, we had to generate scripts within SQL, using generate scripts, and in the advanced options we used the 'Type of Data to script: Schema and Data' option to generate a massive text file. We were successfully able to move our test R2 databases to our legacy SQL 2008 servers -- where a database restore to a lower version would not have worked. We used sqlcmd to input the large file--as the files were often too big for the SSMS text buffer.
What I'm saying here is that this option would probably work for you as well. You'll just need to do one additional step and search and replace getdate() with [dbo].getlocaldate in the generated text file. (I would put your function into the database before migration though).
(I never wanted to be proficient at this band-aid of a database restore, but for a while it became a defacto way of doing things. And, it worked every time.)
If you choose this route, be sure and select the Advanced button and select all the options you need (read each one) to move from the old database to the new database--like the defaults you mentioned. But give it a few test runs in Azure. I bet you'll find that this is one solution that works -- with a modicum of effort.
This is what I'd probably do. I'd have the most faith in this. Or do a generate before and after a programmatic fix to eyeball all the changes
– Paul
Jul 11 '18 at 7:48
add a comment |
I really like David's answer and upvoted that for a programmatic way of doing things.
But you can try this today for a test-run in Azure via SSMS:
Right click your database --> Tasks --> Generate Scripts..
[Back Story] we had a junior DBA who upgraded all our test environments to SQL 2008 R2 while our production environments were at SQL 2008. It's a change that makes me cringe to this day. To migrate to production, from test, we had to generate scripts within SQL, using generate scripts, and in the advanced options we used the 'Type of Data to script: Schema and Data' option to generate a massive text file. We were successfully able to move our test R2 databases to our legacy SQL 2008 servers -- where a database restore to a lower version would not have worked. We used sqlcmd to input the large file--as the files were often too big for the SSMS text buffer.
What I'm saying here is that this option would probably work for you as well. You'll just need to do one additional step and search and replace getdate() with [dbo].getlocaldate in the generated text file. (I would put your function into the database before migration though).
(I never wanted to be proficient at this band-aid of a database restore, but for a while it became a defacto way of doing things. And, it worked every time.)
If you choose this route, be sure and select the Advanced button and select all the options you need (read each one) to move from the old database to the new database--like the defaults you mentioned. But give it a few test runs in Azure. I bet you'll find that this is one solution that works -- with a modicum of effort.
I really like David's answer and upvoted that for a programmatic way of doing things.
But you can try this today for a test-run in Azure via SSMS:
Right click your database --> Tasks --> Generate Scripts..
[Back Story] we had a junior DBA who upgraded all our test environments to SQL 2008 R2 while our production environments were at SQL 2008. It's a change that makes me cringe to this day. To migrate to production, from test, we had to generate scripts within SQL, using generate scripts, and in the advanced options we used the 'Type of Data to script: Schema and Data' option to generate a massive text file. We were successfully able to move our test R2 databases to our legacy SQL 2008 servers -- where a database restore to a lower version would not have worked. We used sqlcmd to input the large file--as the files were often too big for the SSMS text buffer.
What I'm saying here is that this option would probably work for you as well. You'll just need to do one additional step and search and replace getdate() with [dbo].getlocaldate in the generated text file. (I would put your function into the database before migration though).
(I never wanted to be proficient at this band-aid of a database restore, but for a while it became a defacto way of doing things. And, it worked every time.)
If you choose this route, be sure and select the Advanced button and select all the options you need (read each one) to move from the old database to the new database--like the defaults you mentioned. But give it a few test runs in Azure. I bet you'll find that this is one solution that works -- with a modicum of effort.
edited Jul 10 '18 at 19:05
answered Jul 10 '18 at 15:49
StingSting
2,212416
2,212416
This is what I'd probably do. I'd have the most faith in this. Or do a generate before and after a programmatic fix to eyeball all the changes
– Paul
Jul 11 '18 at 7:48
add a comment |
This is what I'd probably do. I'd have the most faith in this. Or do a generate before and after a programmatic fix to eyeball all the changes
– Paul
Jul 11 '18 at 7:48
This is what I'd probably do. I'd have the most faith in this. Or do a generate before and after a programmatic fix to eyeball all the changes
– Paul
Jul 11 '18 at 7:48
This is what I'd probably do. I'd have the most faith in this. Or do a generate before and after a programmatic fix to eyeball all the changes
– Paul
Jul 11 '18 at 7:48
add a comment |
Dynamically alter all proc and udf to change value
DECLARE @Text NVARCHAR(max),
@spname NVARCHAR(max),
@Type CHAR(5),
@Sql NVARCHAR(max)
DECLARE @getobject CURSOR
SET @getobject = CURSOR
FOR SELECT sc.text,
so.NAME,
so.type
FROM sys.syscomments sc
INNER JOIN sysobjects so
ON sc.id = so.id
WHERE sc.[text] LIKE '%getdate()%'
--and type in('P','FN')
OPEN @getobject
FETCH next FROM @getobject INTO @Text, @spname, @Type
WHILE @@FETCH_STATUS = 0
BEGIN
IF ( @Type = 'P'
OR @Type = 'FN' )
SET @Text = Replace(@Text, 'getdate', 'dbo.getlocaldate')
SET @Text = Replace(@Text, 'create', 'alter')
EXECUTE Sp_executesql
@Text
PRINT @Text
--,@spname,@Type
FETCH next FROM @getobject INTO @Text, @spname, @Type
END
CLOSE @getobject
DEALLOCATE @getobject
CREATE PROCEDURE [dbo].[Testproc1]
AS
SET nocount ON;
BEGIN
DECLARE @CurDate DATETIME = Getdate()
END
Notice commented sysobjects Type column condition.My script will alter only proc and UDF.
This script will alter all Default Constraint
which contain GetDate()
DECLARE @TableName VARCHAR(300),
@constraintName VARCHAR(300),
@colName VARCHAR(300),
@Sql NVARCHAR(max)
DECLARE @getobject CURSOR
SET @getobject = CURSOR
FOR SELECT ds.NAME,
sc.NAME AS colName,
so.NAME AS Tablename
--,ds.definition
FROM sys.default_constraints ds
INNER JOIN sys.columns sc
ON ds.object_id = sc.default_object_id
INNER JOIN sys.objects so
ON so.object_id = ds.parent_object_id
WHERE definition LIKE '%getdate()%'
OPEN @getobject
FETCH next FROM @getobject INTO @constraintName, @colName, @TableName
WHILE @@FETCH_STATUS = 0
BEGIN
SET @Sql = 'ALTER TABLE ' + @TableName
+ ' DROP CONSTRAINT ' + @constraintName + '; '
+ Char(13) + Char(10) + ' ' + Char(13) + Char(10) + ''
SET @Sql = @Sql + ' ALTER TABLE ' + @TableName
+ ' ADD CONSTRAINT ' + @constraintName
+ ' DEFAULT dbo.GetLocaledate() FOR '
+ @colName + ';' + Char(13) + Char(10) + ' ' + Char(13)
+ Char(10) + ''
PRINT @Sql
EXECUTE sys.Sp_executesql
@Sql
--,@spname,@Type
FETCH next FROM @getobject INTO @constraintName, @colName, @TableName
END
CLOSE @getobject
DEALLOCATE @getobject
add a comment |
Dynamically alter all proc and udf to change value
DECLARE @Text NVARCHAR(max),
@spname NVARCHAR(max),
@Type CHAR(5),
@Sql NVARCHAR(max)
DECLARE @getobject CURSOR
SET @getobject = CURSOR
FOR SELECT sc.text,
so.NAME,
so.type
FROM sys.syscomments sc
INNER JOIN sysobjects so
ON sc.id = so.id
WHERE sc.[text] LIKE '%getdate()%'
--and type in('P','FN')
OPEN @getobject
FETCH next FROM @getobject INTO @Text, @spname, @Type
WHILE @@FETCH_STATUS = 0
BEGIN
IF ( @Type = 'P'
OR @Type = 'FN' )
SET @Text = Replace(@Text, 'getdate', 'dbo.getlocaldate')
SET @Text = Replace(@Text, 'create', 'alter')
EXECUTE Sp_executesql
@Text
PRINT @Text
--,@spname,@Type
FETCH next FROM @getobject INTO @Text, @spname, @Type
END
CLOSE @getobject
DEALLOCATE @getobject
CREATE PROCEDURE [dbo].[Testproc1]
AS
SET nocount ON;
BEGIN
DECLARE @CurDate DATETIME = Getdate()
END
Notice commented sysobjects Type column condition.My script will alter only proc and UDF.
This script will alter all Default Constraint
which contain GetDate()
DECLARE @TableName VARCHAR(300),
@constraintName VARCHAR(300),
@colName VARCHAR(300),
@Sql NVARCHAR(max)
DECLARE @getobject CURSOR
SET @getobject = CURSOR
FOR SELECT ds.NAME,
sc.NAME AS colName,
so.NAME AS Tablename
--,ds.definition
FROM sys.default_constraints ds
INNER JOIN sys.columns sc
ON ds.object_id = sc.default_object_id
INNER JOIN sys.objects so
ON so.object_id = ds.parent_object_id
WHERE definition LIKE '%getdate()%'
OPEN @getobject
FETCH next FROM @getobject INTO @constraintName, @colName, @TableName
WHILE @@FETCH_STATUS = 0
BEGIN
SET @Sql = 'ALTER TABLE ' + @TableName
+ ' DROP CONSTRAINT ' + @constraintName + '; '
+ Char(13) + Char(10) + ' ' + Char(13) + Char(10) + ''
SET @Sql = @Sql + ' ALTER TABLE ' + @TableName
+ ' ADD CONSTRAINT ' + @constraintName
+ ' DEFAULT dbo.GetLocaledate() FOR '
+ @colName + ';' + Char(13) + Char(10) + ' ' + Char(13)
+ Char(10) + ''
PRINT @Sql
EXECUTE sys.Sp_executesql
@Sql
--,@spname,@Type
FETCH next FROM @getobject INTO @constraintName, @colName, @TableName
END
CLOSE @getobject
DEALLOCATE @getobject
add a comment |
Dynamically alter all proc and udf to change value
DECLARE @Text NVARCHAR(max),
@spname NVARCHAR(max),
@Type CHAR(5),
@Sql NVARCHAR(max)
DECLARE @getobject CURSOR
SET @getobject = CURSOR
FOR SELECT sc.text,
so.NAME,
so.type
FROM sys.syscomments sc
INNER JOIN sysobjects so
ON sc.id = so.id
WHERE sc.[text] LIKE '%getdate()%'
--and type in('P','FN')
OPEN @getobject
FETCH next FROM @getobject INTO @Text, @spname, @Type
WHILE @@FETCH_STATUS = 0
BEGIN
IF ( @Type = 'P'
OR @Type = 'FN' )
SET @Text = Replace(@Text, 'getdate', 'dbo.getlocaldate')
SET @Text = Replace(@Text, 'create', 'alter')
EXECUTE Sp_executesql
@Text
PRINT @Text
--,@spname,@Type
FETCH next FROM @getobject INTO @Text, @spname, @Type
END
CLOSE @getobject
DEALLOCATE @getobject
CREATE PROCEDURE [dbo].[Testproc1]
AS
SET nocount ON;
BEGIN
DECLARE @CurDate DATETIME = Getdate()
END
Notice commented sysobjects Type column condition.My script will alter only proc and UDF.
This script will alter all Default Constraint
which contain GetDate()
DECLARE @TableName VARCHAR(300),
@constraintName VARCHAR(300),
@colName VARCHAR(300),
@Sql NVARCHAR(max)
DECLARE @getobject CURSOR
SET @getobject = CURSOR
FOR SELECT ds.NAME,
sc.NAME AS colName,
so.NAME AS Tablename
--,ds.definition
FROM sys.default_constraints ds
INNER JOIN sys.columns sc
ON ds.object_id = sc.default_object_id
INNER JOIN sys.objects so
ON so.object_id = ds.parent_object_id
WHERE definition LIKE '%getdate()%'
OPEN @getobject
FETCH next FROM @getobject INTO @constraintName, @colName, @TableName
WHILE @@FETCH_STATUS = 0
BEGIN
SET @Sql = 'ALTER TABLE ' + @TableName
+ ' DROP CONSTRAINT ' + @constraintName + '; '
+ Char(13) + Char(10) + ' ' + Char(13) + Char(10) + ''
SET @Sql = @Sql + ' ALTER TABLE ' + @TableName
+ ' ADD CONSTRAINT ' + @constraintName
+ ' DEFAULT dbo.GetLocaledate() FOR '
+ @colName + ';' + Char(13) + Char(10) + ' ' + Char(13)
+ Char(10) + ''
PRINT @Sql
EXECUTE sys.Sp_executesql
@Sql
--,@spname,@Type
FETCH next FROM @getobject INTO @constraintName, @colName, @TableName
END
CLOSE @getobject
DEALLOCATE @getobject
Dynamically alter all proc and udf to change value
DECLARE @Text NVARCHAR(max),
@spname NVARCHAR(max),
@Type CHAR(5),
@Sql NVARCHAR(max)
DECLARE @getobject CURSOR
SET @getobject = CURSOR
FOR SELECT sc.text,
so.NAME,
so.type
FROM sys.syscomments sc
INNER JOIN sysobjects so
ON sc.id = so.id
WHERE sc.[text] LIKE '%getdate()%'
--and type in('P','FN')
OPEN @getobject
FETCH next FROM @getobject INTO @Text, @spname, @Type
WHILE @@FETCH_STATUS = 0
BEGIN
IF ( @Type = 'P'
OR @Type = 'FN' )
SET @Text = Replace(@Text, 'getdate', 'dbo.getlocaldate')
SET @Text = Replace(@Text, 'create', 'alter')
EXECUTE Sp_executesql
@Text
PRINT @Text
--,@spname,@Type
FETCH next FROM @getobject INTO @Text, @spname, @Type
END
CLOSE @getobject
DEALLOCATE @getobject
CREATE PROCEDURE [dbo].[Testproc1]
AS
SET nocount ON;
BEGIN
DECLARE @CurDate DATETIME = Getdate()
END
Notice commented sysobjects Type column condition.My script will alter only proc and UDF.
This script will alter all Default Constraint
which contain GetDate()
DECLARE @TableName VARCHAR(300),
@constraintName VARCHAR(300),
@colName VARCHAR(300),
@Sql NVARCHAR(max)
DECLARE @getobject CURSOR
SET @getobject = CURSOR
FOR SELECT ds.NAME,
sc.NAME AS colName,
so.NAME AS Tablename
--,ds.definition
FROM sys.default_constraints ds
INNER JOIN sys.columns sc
ON ds.object_id = sc.default_object_id
INNER JOIN sys.objects so
ON so.object_id = ds.parent_object_id
WHERE definition LIKE '%getdate()%'
OPEN @getobject
FETCH next FROM @getobject INTO @constraintName, @colName, @TableName
WHILE @@FETCH_STATUS = 0
BEGIN
SET @Sql = 'ALTER TABLE ' + @TableName
+ ' DROP CONSTRAINT ' + @constraintName + '; '
+ Char(13) + Char(10) + ' ' + Char(13) + Char(10) + ''
SET @Sql = @Sql + ' ALTER TABLE ' + @TableName
+ ' ADD CONSTRAINT ' + @constraintName
+ ' DEFAULT dbo.GetLocaledate() FOR '
+ @colName + ';' + Char(13) + Char(10) + ' ' + Char(13)
+ Char(10) + ''
PRINT @Sql
EXECUTE sys.Sp_executesql
@Sql
--,@spname,@Type
FETCH next FROM @getobject INTO @constraintName, @colName, @TableName
END
CLOSE @getobject
DEALLOCATE @getobject
edited Jul 13 '18 at 11:31
answered Jul 13 '18 at 10:35
KumarHarshKumarHarsh
81358
81358
add a comment |
add a comment |
I have upvoted Evan Carrolls answer, as I think this is the best solution. I have not been able to convince my colleagues that they should change a lot of C# code, so I had to use the code that David Spillett wrote. I have fixed a couple of problems with Dynamic SQL and Schemas (not all code use "dbo.") like this:
DECLARE C CURSOR LOCAL STATIC FOR
SELECT sm.definition, so.type
FROM sys.objects so
JOIN sys.all_sql_modules sm ON sm.object_id = so.object_id
WHERE so.type IN ('P', 'V')
AND CHARINDEX('getdate()', sm.definition) > 0
ORDER BY so.name
DECLARE @SQL NVARCHAR(MAX), @objtype NVARCHAR(MAX)
OPEN C
WHILE 1=1 BEGIN
FETCH NEXT FROM C INTO @SQL, @objtype
IF @@FETCH_STATUS <> 0 BREAK
IF @objtype = 'P' SET @SQL = REPLACE(@SQL, 'CREATE PROCEDURE', 'ALTER PROCEDURE')
IF @objtype = 'P' SET @SQL = REPLACE(@SQL, 'CREATE PROCEDURE', 'ALTER PROCEDURE') /* when you write "create or alter proc" */
IF @objtype = 'V' SET @SQL = REPLACE(@SQL, 'CREATE VIEW' , 'ALTER VIEW' )
IF CHARINDEX('getdate())''', @sql) > 0 BEGIN /* when dynamic SQL is used */
IF CHARINDEX('utl.getdate())''', @sql) = 0 SET @SQL = REPLACE(@SQL, 'GETDATE()', 'utl.getdate()')
end
ELSE begin
SET @SQL = REPLACE(@SQL, 'GETDATE()', 'CONVERT(DATETIME, CONVERT(datetimeoffset, SYSDATETIME()) AT TIME ZONE ''Central Europe Standard Time'')')
end
EXEC dbo.LongPrint @String = @sql
EXEC (@SQL)
END
CLOSE C
DEALLOCATE C
and the default constraints like this:
DECLARE C CURSOR LOCAL STATIC FOR
SELECT AlterDefaultSQL = 'ALTER TABLE [' +sch.name+ '].[' +st.name+ '] DROP CONSTRAINT [' + si.name + '];'
+ CHAR(10)
+ 'ALTER TABLE [' +sch.name+ '].[' +st.name+ '] ADD CONSTRAINT [' + si.name + '] DEFAULT '+REPLACE(si.definition, 'GETDATE()', 'CONVERT(DATETIME, CONVERT(datetimeoffset, SYSDATETIME()) AT TIME ZONE ''Central Europe Standard Time'')')+' FOR '+sc.name+';'
FROM sys.tables st
JOIN sys.default_constraints si ON si.parent_object_id = st.object_id
JOIN sys.columns sc ON sc.default_object_id = si.object_id
INNER JOIN sys.schemas sch ON sch.schema_id = st.schema_id
WHERE CHARINDEX('getdate()', si.definition) > 0
ORDER BY st.name, sc.name
DECLARE @SQL NVARCHAR(MAX)
OPEN C
WHILE 1=1 BEGIN
FETCH NEXT FROM C INTO @SQL
IF @@FETCH_STATUS <> 0 BREAK
EXEC dbo.LongPrint @String = @sql
EXEC (@SQL)
FETCH NEXT FROM C INTO @SQL
END
CLOSE C
DEALLOCATE C
add a comment |
I have upvoted Evan Carrolls answer, as I think this is the best solution. I have not been able to convince my colleagues that they should change a lot of C# code, so I had to use the code that David Spillett wrote. I have fixed a couple of problems with Dynamic SQL and Schemas (not all code use "dbo.") like this:
DECLARE C CURSOR LOCAL STATIC FOR
SELECT sm.definition, so.type
FROM sys.objects so
JOIN sys.all_sql_modules sm ON sm.object_id = so.object_id
WHERE so.type IN ('P', 'V')
AND CHARINDEX('getdate()', sm.definition) > 0
ORDER BY so.name
DECLARE @SQL NVARCHAR(MAX), @objtype NVARCHAR(MAX)
OPEN C
WHILE 1=1 BEGIN
FETCH NEXT FROM C INTO @SQL, @objtype
IF @@FETCH_STATUS <> 0 BREAK
IF @objtype = 'P' SET @SQL = REPLACE(@SQL, 'CREATE PROCEDURE', 'ALTER PROCEDURE')
IF @objtype = 'P' SET @SQL = REPLACE(@SQL, 'CREATE PROCEDURE', 'ALTER PROCEDURE') /* when you write "create or alter proc" */
IF @objtype = 'V' SET @SQL = REPLACE(@SQL, 'CREATE VIEW' , 'ALTER VIEW' )
IF CHARINDEX('getdate())''', @sql) > 0 BEGIN /* when dynamic SQL is used */
IF CHARINDEX('utl.getdate())''', @sql) = 0 SET @SQL = REPLACE(@SQL, 'GETDATE()', 'utl.getdate()')
end
ELSE begin
SET @SQL = REPLACE(@SQL, 'GETDATE()', 'CONVERT(DATETIME, CONVERT(datetimeoffset, SYSDATETIME()) AT TIME ZONE ''Central Europe Standard Time'')')
end
EXEC dbo.LongPrint @String = @sql
EXEC (@SQL)
END
CLOSE C
DEALLOCATE C
and the default constraints like this:
DECLARE C CURSOR LOCAL STATIC FOR
SELECT AlterDefaultSQL = 'ALTER TABLE [' +sch.name+ '].[' +st.name+ '] DROP CONSTRAINT [' + si.name + '];'
+ CHAR(10)
+ 'ALTER TABLE [' +sch.name+ '].[' +st.name+ '] ADD CONSTRAINT [' + si.name + '] DEFAULT '+REPLACE(si.definition, 'GETDATE()', 'CONVERT(DATETIME, CONVERT(datetimeoffset, SYSDATETIME()) AT TIME ZONE ''Central Europe Standard Time'')')+' FOR '+sc.name+';'
FROM sys.tables st
JOIN sys.default_constraints si ON si.parent_object_id = st.object_id
JOIN sys.columns sc ON sc.default_object_id = si.object_id
INNER JOIN sys.schemas sch ON sch.schema_id = st.schema_id
WHERE CHARINDEX('getdate()', si.definition) > 0
ORDER BY st.name, sc.name
DECLARE @SQL NVARCHAR(MAX)
OPEN C
WHILE 1=1 BEGIN
FETCH NEXT FROM C INTO @SQL
IF @@FETCH_STATUS <> 0 BREAK
EXEC dbo.LongPrint @String = @sql
EXEC (@SQL)
FETCH NEXT FROM C INTO @SQL
END
CLOSE C
DEALLOCATE C
add a comment |
I have upvoted Evan Carrolls answer, as I think this is the best solution. I have not been able to convince my colleagues that they should change a lot of C# code, so I had to use the code that David Spillett wrote. I have fixed a couple of problems with Dynamic SQL and Schemas (not all code use "dbo.") like this:
DECLARE C CURSOR LOCAL STATIC FOR
SELECT sm.definition, so.type
FROM sys.objects so
JOIN sys.all_sql_modules sm ON sm.object_id = so.object_id
WHERE so.type IN ('P', 'V')
AND CHARINDEX('getdate()', sm.definition) > 0
ORDER BY so.name
DECLARE @SQL NVARCHAR(MAX), @objtype NVARCHAR(MAX)
OPEN C
WHILE 1=1 BEGIN
FETCH NEXT FROM C INTO @SQL, @objtype
IF @@FETCH_STATUS <> 0 BREAK
IF @objtype = 'P' SET @SQL = REPLACE(@SQL, 'CREATE PROCEDURE', 'ALTER PROCEDURE')
IF @objtype = 'P' SET @SQL = REPLACE(@SQL, 'CREATE PROCEDURE', 'ALTER PROCEDURE') /* when you write "create or alter proc" */
IF @objtype = 'V' SET @SQL = REPLACE(@SQL, 'CREATE VIEW' , 'ALTER VIEW' )
IF CHARINDEX('getdate())''', @sql) > 0 BEGIN /* when dynamic SQL is used */
IF CHARINDEX('utl.getdate())''', @sql) = 0 SET @SQL = REPLACE(@SQL, 'GETDATE()', 'utl.getdate()')
end
ELSE begin
SET @SQL = REPLACE(@SQL, 'GETDATE()', 'CONVERT(DATETIME, CONVERT(datetimeoffset, SYSDATETIME()) AT TIME ZONE ''Central Europe Standard Time'')')
end
EXEC dbo.LongPrint @String = @sql
EXEC (@SQL)
END
CLOSE C
DEALLOCATE C
and the default constraints like this:
DECLARE C CURSOR LOCAL STATIC FOR
SELECT AlterDefaultSQL = 'ALTER TABLE [' +sch.name+ '].[' +st.name+ '] DROP CONSTRAINT [' + si.name + '];'
+ CHAR(10)
+ 'ALTER TABLE [' +sch.name+ '].[' +st.name+ '] ADD CONSTRAINT [' + si.name + '] DEFAULT '+REPLACE(si.definition, 'GETDATE()', 'CONVERT(DATETIME, CONVERT(datetimeoffset, SYSDATETIME()) AT TIME ZONE ''Central Europe Standard Time'')')+' FOR '+sc.name+';'
FROM sys.tables st
JOIN sys.default_constraints si ON si.parent_object_id = st.object_id
JOIN sys.columns sc ON sc.default_object_id = si.object_id
INNER JOIN sys.schemas sch ON sch.schema_id = st.schema_id
WHERE CHARINDEX('getdate()', si.definition) > 0
ORDER BY st.name, sc.name
DECLARE @SQL NVARCHAR(MAX)
OPEN C
WHILE 1=1 BEGIN
FETCH NEXT FROM C INTO @SQL
IF @@FETCH_STATUS <> 0 BREAK
EXEC dbo.LongPrint @String = @sql
EXEC (@SQL)
FETCH NEXT FROM C INTO @SQL
END
CLOSE C
DEALLOCATE C
I have upvoted Evan Carrolls answer, as I think this is the best solution. I have not been able to convince my colleagues that they should change a lot of C# code, so I had to use the code that David Spillett wrote. I have fixed a couple of problems with Dynamic SQL and Schemas (not all code use "dbo.") like this:
DECLARE C CURSOR LOCAL STATIC FOR
SELECT sm.definition, so.type
FROM sys.objects so
JOIN sys.all_sql_modules sm ON sm.object_id = so.object_id
WHERE so.type IN ('P', 'V')
AND CHARINDEX('getdate()', sm.definition) > 0
ORDER BY so.name
DECLARE @SQL NVARCHAR(MAX), @objtype NVARCHAR(MAX)
OPEN C
WHILE 1=1 BEGIN
FETCH NEXT FROM C INTO @SQL, @objtype
IF @@FETCH_STATUS <> 0 BREAK
IF @objtype = 'P' SET @SQL = REPLACE(@SQL, 'CREATE PROCEDURE', 'ALTER PROCEDURE')
IF @objtype = 'P' SET @SQL = REPLACE(@SQL, 'CREATE PROCEDURE', 'ALTER PROCEDURE') /* when you write "create or alter proc" */
IF @objtype = 'V' SET @SQL = REPLACE(@SQL, 'CREATE VIEW' , 'ALTER VIEW' )
IF CHARINDEX('getdate())''', @sql) > 0 BEGIN /* when dynamic SQL is used */
IF CHARINDEX('utl.getdate())''', @sql) = 0 SET @SQL = REPLACE(@SQL, 'GETDATE()', 'utl.getdate()')
end
ELSE begin
SET @SQL = REPLACE(@SQL, 'GETDATE()', 'CONVERT(DATETIME, CONVERT(datetimeoffset, SYSDATETIME()) AT TIME ZONE ''Central Europe Standard Time'')')
end
EXEC dbo.LongPrint @String = @sql
EXEC (@SQL)
END
CLOSE C
DEALLOCATE C
and the default constraints like this:
DECLARE C CURSOR LOCAL STATIC FOR
SELECT AlterDefaultSQL = 'ALTER TABLE [' +sch.name+ '].[' +st.name+ '] DROP CONSTRAINT [' + si.name + '];'
+ CHAR(10)
+ 'ALTER TABLE [' +sch.name+ '].[' +st.name+ '] ADD CONSTRAINT [' + si.name + '] DEFAULT '+REPLACE(si.definition, 'GETDATE()', 'CONVERT(DATETIME, CONVERT(datetimeoffset, SYSDATETIME()) AT TIME ZONE ''Central Europe Standard Time'')')+' FOR '+sc.name+';'
FROM sys.tables st
JOIN sys.default_constraints si ON si.parent_object_id = st.object_id
JOIN sys.columns sc ON sc.default_object_id = si.object_id
INNER JOIN sys.schemas sch ON sch.schema_id = st.schema_id
WHERE CHARINDEX('getdate()', si.definition) > 0
ORDER BY st.name, sc.name
DECLARE @SQL NVARCHAR(MAX)
OPEN C
WHILE 1=1 BEGIN
FETCH NEXT FROM C INTO @SQL
IF @@FETCH_STATUS <> 0 BREAK
EXEC dbo.LongPrint @String = @sql
EXEC (@SQL)
FETCH NEXT FROM C INTO @SQL
END
CLOSE C
DEALLOCATE C
answered 7 mins ago
Henrik Staun PoulsenHenrik Staun Poulsen
1,1121823
1,1121823
add a comment |
add a comment |
Thanks for contributing an answer to Database Administrators Stack Exchange!
- Please be sure to answer the question. Provide details and share your research!
But avoid …
- Asking for help, clarification, or responding to other answers.
- Making statements based on opinion; back them up with references or personal experience.
To learn more, see our tips on writing great answers.
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
StackExchange.ready(
function () {
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fdba.stackexchange.com%2fquestions%2f211352%2fchanging-the-use-of-getdate-in-the-entire-database%23new-answer', 'question_page');
}
);
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
5
It might sound daft, but importing the database in an SSDT project, replacing the getdate() function with a custom one and then creating the new function as per your needs might work.
– Marian
Jul 4 '18 at 17:23
What problem are you having implementing the change?
– Max Vernon
Jul 4 '18 at 18:12
Sounds like you need to figure out a query that will bring back results for all objects
GETDATE()
is referenced and then figure out ways to make those change and then just keep looking over the query list until you got them all. Perhaps you can get most of them changed with one of the method below and then do the others manually. I would think even if 50% - 80% are completed via a process then the other 50% - 20% needs to be done manually, that's better than nothing, correct? I'd start with something and start getting it done for sure.– Pimp Juice IT
Jul 10 '18 at 3:40