Re: [PATCH] pgarchives: Fix database install procedure: remove redundant tables in schema.sql

From: Célestin Matte <celestin(dot)matte(at)cmatte(dot)me>
To: pgsql-www(at)lists(dot)postgresql(dot)org
Subject: Re: [PATCH] pgarchives: Fix database install procedure: remove redundant tables in schema.sql
Date: 2021-10-22 15:19:57
Message-ID: 71f7adb0-8588-a695-acad-f7034d3315d4@cmatte.me
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-www

Hello,

Apologies, patch sent in previous email of this thread is incorrect. Please find correct patch attached.

In fact, the pgarchives repository is in a weird state, in which the migration adding two tables is missing (0005_apiclient_threadsubscription.py when running makemigrations). My guess is that this is because these tables use SERIALs, which django does not handle properly, and must by created using raw SQL. Am I right?
Is it something that should be fixed?

I would still need answers for the two questions asked in previous email:

> 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.
>
> Cheers,
>

Cheers,
--
Célestin Matte

Attachment Content-Type Size
0001-Fix-database-install-procedure-remove-redundant-tabl.patch text/x-patch 2.6 KB

In response to

Responses

Browse pgsql-www by date

  From Date Subject
Next Message Jonathan S. Katz 2021-10-22 16:10:48 Re: [PATCH] pgweb: Add SEARCH_DSN to example settings.py
Previous Message Célestin Matte 2021-10-22 15:08:19 [PATCH] pgweb: Add SEARCH_DSN to example settings.py