You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Expected Behavior
Migrations should not need to run SET SCHEMA. Instead, the Postgres db driver should handle that for you
Migrate Version
v4.9.1
Loaded Source Drivers
N/A
Loaded Database Drivers
N/A
Go Version
1.14
Stacktrace
N/A
Additional context
N/A
Potential Solutions
I think we can add a method (ensureUsingSchema()) to the postgres in a backwards compatible manner. Not sure if we should run ensureUsingSchema() before every migration since that may break existing migrations, we could run it when the connection opens. That way any exist SET SCHEMA call is honored and subsequent migrations would continue to use the set schema.
Testcases
Any fix should test the following cases:
Where schema specified:
In config
Not in config
In migration
Not in migration
The text was updated successfully, but these errors were encountered:
Describe the Bug
It looks like the schema is only used for generating the advisory lock id and for dropping migrations.
Steps to Reproduce
See: #262 (comment)
Expected Behavior
Migrations should not need to run
SET SCHEMA
. Instead, the Postgres db driver should handle that for youMigrate Version
v4.9.1
Loaded Source Drivers
N/A
Loaded Database Drivers
N/A
Go Version
1.14
Stacktrace
N/A
Additional context
N/A
Potential Solutions
ensureUsingSchema()
) to the postgres in a backwards compatible manner. Not sure if we should runensureUsingSchema()
before every migration since that may break existing migrations, we could run it when the connection opens. That way any existSET SCHEMA
call is honored and subsequent migrations would continue to use the set schema.Testcases
Any fix should test the following cases:
The text was updated successfully, but these errors were encountered: