From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Christophe Courtois <christophe(dot)courtois(at)dalibo(dot)com> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-docs(at)postgresql(dot)org |
Subject: | Re: [DOCS] intagg.sgml: example wrongly named and does not compile |
Date: | 2023-11-22 03:27:22 |
Message-ID: | ZV11Gh7fB2G39jR9@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-docs |
On Fri, Apr 21, 2017 at 06:07:13PM +0200, Christophe Courtois wrote:
> Le 21/04/2017 à 17:45, Tom Lane a écrit :
> > Christophe Courtois <christophe(dot)courtois(at)dalibo(dot)com> writes:
> >> I've found out that the example in intagg.sgml is wrongly named: the
> >> one-to-many table is a many-to-many.
> > Well, it'd depend on how it was used. The example clearly intends that
> > it be one-to-many, and I'm not sure it still makes sense without that
> > restriction. Maybe better to add a unique constraint on
> > one_to_many(left)?
>
> Perhaps the whole example can be simplified to get rid of the "left"
> table, but I didn't intend to rewrite it.
>
> >> And my colleague Thibaut Madeleine has seen that the "CREATE TABLE
> >> right" and "CREATE TABLE left" examples cannot compile due to the
> >> reserved words.
> > Ouch. Shows you how old this module is :-(
>
> Indeed.
>
> >> I propose the attached patch to fix that.
> > Um, the attached file seems empty from here.
>
> Ooops, sorry. It is attached.
I like this six year old patch so would like to apply it to master,
attached.
--
Bruce Momjian <bruce(at)momjian(dot)us> https://momjian.us
EDB https://enterprisedb.com
Only you can decide what is important to you.
Attachment | Content-Type | Size |
---|---|---|
agg.diff | text/x-diff | 4.1 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2023-11-22 04:33:39 | Re: [DOCS] Add example about date ISO format |
Previous Message | Bruce Momjian | 2023-11-22 02:36:31 | Re: [DOCS] The reference to 'atacontrol' on FreeBSD is outdated. |