We are using liquibase formatted SQL (and need to stay in this format for the time being) and it has been going well, but I just encountered an issue that I cannot figure out how to get around.
We have had a lot of `IF EXISTS (proc) DROP PROC` statements and those seem to work fine.
But, one of the developers just added a check for a column on a table; the SQL executes properly directly against MSSQL but fails when ran as Liquibase.
The logic is if the table contains a given column, then update the values in that column. Liquibase appears to be properly capturing the result of the IF statement, but it seems it still tries to evaluate the validity of the condition inside of it.
Here is the desired Query:
- -- changeset damon.overboe:test-if-exists-3 endDelimiter:\nGO runOnChange:true
- IF EXISTS (SELECT * FROM INFORMATION_SCHEMA.columns WHERE TABLE_NAME='good_table_name AND COLUMN_NAME='bad_column_name')
- BEGIN
- UPDATE good_table_name SET bad_column_name = 1
- END
- GO
I've tried with `BEGIN` and `BEGIN;` and both give the same output:
Reason: liquibase.exception.DatabaseException: Error executing SQL IF ...
%< ----- snip ---- >%
Invalid column name 'bad_column_name'.
I also tried using
- IF (SELECT count(*) FROM INFORMATION_SCHEMA.columns WHERE TABLE_NAME='good_table_name' AND COLUMN_NAME='bad_column_name') <> 0
- ...
And get the same result.
And I have tried dropping the BEGIN and END blocks:
- -- changeset damon.overboe:test-if-exists-4 endDelimiter:\nGO runOnChange:true
- IF EXISTS (SELECT * FROM INFORMATION_SCHEMA.columns WHERE TABLE_NAME='good_table_name AND COLUMN_NAME='bad_column_name')
- UPDATE good_table_name SET bad_column_name = 1
- GO
So, to prove it's not the IF statement's syntax, I added two tables, damon_test_table_should_persist and damon_test_you_shouldnt_see_me, then ran the following two changesets
- -- changeset damon.overboe:test-if-exists-1 endDelimiter:\nGO runOnChange:true
- IF EXISTS (SELECT * FROM INFORMATION_SCHEMA.columns WHERE TABLE_NAME='good_table_name' AND COLUMN_NAME='bad_column_name')
- BEGIN
- -- IF should be false, so this table should not be dropped
DROP TABLE damon_test_table_should_persist
- END
- GO
- -- changeset damon.overboe:test-if-exists-2 endDelimiter:\nGO runOnChange:true
- IF NOT EXISTS (SELECT * FROM INFORMATION_SCHEMA.columns WHERE TABLE_NAME='good_table_name' AND COLUMN_NAME='bad_column_name')
- BEGIN
- -- IF should be true, so this table should be dropped
DROP TABLE damon_test_you_shouldnt_see_me
- END
- GO
Both of those ran successfully, so I don't think Liquibase is having an issue with the syntax, I think it's just evaluating the code inside the IF block.
This command errs as expected (table does not exist, it was properly dropped):
- $ sqlcmd -Q "use mydb; select * from damon_test_you_shouldnt_see_me"
- Changed database context to 'mydb'.
- Msg 208, Level 16, State 1, Server *****, Line 1
- Invalid object name 'damon_test_you_shouldnt_see_me'.
and this command passes as expected (table does exist):
- $ sqlcmd -Q "use mydb; select * from damon_test_table_should_persist"
- Changed database context to 'mydb'.
- ID
- -----------
- (0 rows affected)
SUMMARY:
It seems that Liquibase is properly evaluating and running the IF blocks when needed, but, it seems it's also diving into the IF blocks and evaluating them, whether they should be ran. How can I avoid this?