Skip to content
This repository has been archived by the owner on Jun 28, 2018. It is now read-only.

First migration issue cockroachdb #320

Open
incognick opened this issue Dec 13, 2017 · 2 comments
Open

First migration issue cockroachdb #320

incognick opened this issue Dec 13, 2017 · 2 comments

Comments

@incognick
Copy link

incognick commented Dec 13, 2017

I'm having trouble running my first migration against a local cockroachdb instance. I created the initial migration. I ran it and I get the following error:

migrate -database postgresql://root@127.0.0.1:26257/main?sslmode=disable -path migrations up error: pq: relation "schema_migrations" does not exist in line 0: INSERT INTO "schema_migrations" (version, dirty) VALUES ($1, $2)

The table exists in the main database:

root@:26257/main> show columns from schema_migrations;
+---------+------+-------+---------+-------------+
|  Field  | Type | Null  | Default |   Indices   |
+---------+------+-------+---------+-------------+
| version | INT  | false | NULL    | {"primary"} |
| dirty   | BOOL | false | NULL    | {}          |
+---------+------+-------+---------+-------------+
(2 rows)
root@:26257/main> SELECT * FROM schema_migrations;
+---------+-------+
| version | dirty |
+---------+-------+
+---------+-------+
(0 rows)

Not sure why this isn't working. I installed using go get.

Thanks for this tool and any help you can provide!

@kevinmartinjos
Copy link

I ran into the same issue as well. On ubuntu 18.01, go 1.10.1.
Installed the cli using the go get command

@elliott-davis
Copy link

I can repro this inside the cockroach sql shell with:

BEGIN TRANSACTION;
TRUNCATE schema_migrations;
INSERT INTO schema_migrations (version, dirty) VALUES (1, true);
COMMIT;

elliott-davis pushed a commit to elliott-davis/migrate that referenced this issue May 22, 2018
This actually isn't a problem for postgres but is a problem for cockroach.
Internally cockroach drops the table when running a truncate which causes the insert to fail inside of this transaction. I added repro steps to mattes/migrate#320 which this PR should close.

Closes: mattes/migrate#320

Signed-off-by: Elliott Davis <elliott@excellent.io>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants