Re: [PATCH] pgarchives: Fix database install procedure: remove redundant tables in schema.sql - Mailing list pgsql-www

From Magnus Hagander
Subject Re: [PATCH] pgarchives: Fix database install procedure: remove redundant tables in schema.sql
Date
Msg-id CABUevEy_i1xAKscMv4KZ0=bE8050bBcQfWaNyjwZZBofZx7JgQ@mail.gmail.com
Whole thread Raw
In response to [PATCH] pgarchives: Fix database install procedure: remove redundant tables in schema.sql  (Célestin Matte <celestin.matte@cmatte.me>)
List pgsql-www


On Mon, Oct 18, 2021 at 6:19 PM Célestin Matte <celestin.matte@cmatte.me> wrote:
Hello,

While installing pgarchives, I've encountered issues that are addressed in this patch.
Django's model and schema.sql both contain tables that are necessary for
the execution of the application, but are mutually exclusive.
This is due to some fields in the "messages" table that cannot be defined in
django, and definition of tables in schema.sql that are already created
by django.

Ugh, yeah that is clearly a mess. I thought we had cleaned that up at some point, but clearly we haven't, and tracking changse has been inconsistent.

I think the correct solution to the problem is to actually get rid of loader/sql/schema.sql and move it all into the django migrations. Trying to keep track of half in one place and half in another is just going to lead to more problems down the road.

So your removal of messages there is a good start, but adding some columns back after the fact in a different file is a big ugh. We should just make all that happen in 0001_initial.py.


Additionally, I have two questions:
- Where is the "tsparser" parser defined? (See commit e05f813b of pgarchives).
It is used in schema.sql, but I haven't found its definition in the
pgarchives, pglister or pgweb repositories. Is it an alias to
pg_catalog.pg_ts_parser?


 
(I have been able to complete the install procedure by reverting e05f813b,
but I have no idea what the consequences are for the application)
- Same question for /usr/share/postgresql/12/tsearch_data/pg_dict.stop
I've found pg_dict.syn in pgweb, but not this file.

This is just a textfile with stop words. interestingly enough the prod version for the archives just has the word "sql" in it -- I think there was a great big plan to do somethign good with that, that turned out to never happen.
 

--

pgsql-www by date:

Previous
From: Magnus Hagander
Date:
Subject: Re: [PATCH] pgweb: Search: bugfix: replace sites.baseurl with sites.hostname
Next
From: Magnus Hagander
Date:
Subject: Re: [PATCH] pgarchives: Fix database install procedure: remove redundant tables in schema.sql