Re: [DOCS] intagg.sgml: example wrongly named and does not compile

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

In response to

Responses

Browse pgsql-docs by date

  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.