Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Cockroach cannot support mixing DML and DDL in transactions #2592

Closed
juliahayward opened this issue Dec 6, 2019 · 1 comment
Closed

Cockroach cannot support mixing DML and DDL in transactions #2592

juliahayward opened this issue Dec 6, 2019 · 1 comment
Labels
d: CockroachDB m: Core r: fixed
Milestone
Flyway 6.1.1

Comments

@juliahayward
Copy link
Contributor

juliahayward commented Dec 6, 2019 •
edited

Reported by CockroachDB: migrations which mix DML and DDL in a transaction are not supported by the database - for example

ALTER TABLE Foo ADD COLUMN Bar int;
UPDATE Foo SET Bar = 1;

will throw a column-not-known error. Their advice is that migrations should never be run in transactions, in order to prevent any edge cases.
@juliahayward juliahayward added this to the Flyway 6.1.1 milestone Dec 6, 2019
@juliahayward
Copy link
Contributor Author

Fixed in v6.1.1

juliahayward added a commit that referenced this issue Dec 9, 2019
@juliahayward
Fixed #2592 Cockroach cannot support mixing DDL/DML in transactions
b74ab8d
dohrayme pushed a commit to dohrayme/flyway that referenced this issue Feb 3, 2020
@juliahayward
Fixed flyway#2592 Cockroach cannot support mixing DDL/DML in transact…
8ee1bfc 
…ions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
d: CockroachDB m: Core r: fixed
Projects
None yet
Milestone
Flyway 6.1.1
Development

No branches or pull requests
2 participants
@alextercete @juliahayward