Invalid comment in backend/migrations for file 20240403155357_drop_dup_idx.sql

[aldoborrero]

Hi,

I've detected the following issue when applying the migrations with Atlas on a postgres database:

ec 20 12:17:18 homebot systemd[1]: digger-backend.service: Consumed 141ms CPU time, 16.3M memory peak, 29.1K incoming IP traffic, 21.1K outgoing IP traffic.
Dec 20 12:17:18 homebot systemd[1]: digger-backend.service: Scheduled restart job, restart counter is at 4.
Dec 20 12:17:18 homebot systemd[1]: Starting Digger...
Dec 20 12:17:18 homebot digger-backend-pre-start[2198280]: Migrating to version 20241114202249 from 20240402110915 (25 migrations in total):
Dec 20 12:17:18 homebot digger-backend-pre-start[2198280]:   -- migrating version 20240403155357
Dec 20 12:17:18 homebot digger-backend-pre-start[2198280]:     -> # drop the duplicate index to fix the next migration of renaming
Dec 20 12:17:18 homebot digger-backend-pre-start[2198280]:        DROP INDEX "public"."idx_digger_job_id";
Dec 20 12:17:18 homebot digger-backend-pre-start[2198280]:     pq: syntax error at or near "#"
Dec 20 12:17:18 homebot digger-backend-pre-start[2198280]:   -------------------------
Dec 20 12:17:18 homebot digger-backend-pre-start[2198280]:   -- 10.844936ms
Dec 20 12:17:18 homebot digger-backend-pre-start[2198280]:   -- 1 migration with errors
Dec 20 12:17:18 homebot digger-backend-pre-start[2198280]:   -- 1 sql statement with errors
Dec 20 12:17:18 homebot digger-backend-pre-start[2198280]: Error: sql/migrate: executing statement "# drop the duplicate index to fix the next migration of renaming\nDROP INDEX \"public\".\"idx_digger_job_id\";" from version "20240403155357": pq: syntax error at or near "#"
Dec 20 12:17:18 homebot digger-backend-pre-start[2198280]: sql/migrate: write revision: pq: current transaction is aborted, commands ignored until end of transaction block

The culprit is the following comment that should be -- rather than #:

digger/backend/migrations/20240403155357_drop_dup_idx.sql

Line 1 in 5e62218

# drop the duplicate index to fix the next migration of renaming