diff options
| author | Tony Finn <git@tonyfinn.com> | 2022-01-22 02:32:13 +0000 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-01-22 02:32:13 +0000 |
| commit | cdc007c83cafba9884470af0167edd8dd70a0b1d (patch) | |
| tree | 6fbf53b909c3b8f7fbe97777082bea6633d6dc59 /config | |
| parent | 8b0ed8b110752327b98565e4ecc4faf437c0bb3b (diff) | |
| download | invidious-cdc007c83cafba9884470af0167edd8dd70a0b1d.tar.gz invidious-cdc007c83cafba9884470af0167edd8dd70a0b1d.tar.bz2 invidious-cdc007c83cafba9884470af0167edd8dd70a0b1d.zip | |
Specify the public schema for public videos
Unlike the other migrations, this one uses a bare table name to specify the table. This means it looks in the default search path. Unfortunately, the out of box search path is:
1. Schema with the same name as the user
2. Public
Because of this, if a schema exists with the same name as the DB user, the playlist_videos table will get created in _that_ schema, rather than the public schema, and then the grant statement will fail with an error like below if check_table is enabled as it keeps trying to create the table but failing on the grant statement:
```
2022-01-22 02:27:42 UTC [info] check_table: check_table: CREATE TABLE playlist_videos
Unhandled exception: relation "public.playlist_videos" does not exist (PQ::PQError)
from /usr/share/crystal/src/primitives.cr:266:3 in 'handle_error'
from lib/pg/src/pq/connection.cr:184:9 in 'handle_async_frames'
from lib/pg/src/pq/connection.cr:162:7 in 'read'
from lib/pg/src/pq/query.cr:53:14 in 'exec_all'
from lib/db/src/db/database.cr:126:9 in '__crystal_main'
from /usr/share/crystal/src/crystal/main.cr:110:5 in 'main'
from src/env/__libc_start_main.c:94:2 in 'libc_start_main_stage2'
2022-01-22 02:27:43 UTC [info] check_table: check_table: CREATE TABLE playlist_videos
Unhandled exception: relation "public.playlist_videos" does not exist (PQ::PQError)
from /usr/share/crystal/src/primitives.cr:266:3 in 'handle_error'
from lib/pg/src/pq/connection.cr:184:9 in 'handle_async_frames'
from lib/pg/src/pq/connection.cr:162:7 in 'read'
from lib/pg/src/pq/query.cr:53:14 in 'exec_all'
from lib/db/src/db/database.cr:126:9 in '__crystal_main'
from /usr/share/crystal/src/crystal/main.cr:110:5 in 'main'
from src/env/__libc_start_main.c:94:2 in 'libc_start_main_stage2'
```
Diffstat (limited to 'config')
| -rw-r--r-- | config/sql/playlist_videos.sql | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/config/sql/playlist_videos.sql b/config/sql/playlist_videos.sql index eedccbad..4b48b46a 100644 --- a/config/sql/playlist_videos.sql +++ b/config/sql/playlist_videos.sql @@ -2,7 +2,7 @@ -- DROP TABLE public.playlist_videos; -CREATE TABLE IF NOT EXISTS playlist_videos +CREATE TABLE IF NOT EXISTS public.playlist_videos ( title text, id text, |