From: | "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com> |
---|---|
To: | Anton Voloshin <a(dot)voloshin(at)postgrespro(dot)ru> |
Cc: | pgsql-docs <pgsql-docs(at)lists(dot)postgresql(dot)org> |
Subject: | Re: three small improvements for "Composite Types" page |
Date: | 2024-04-12 20:44:59 |
Message-ID: | CAKFQuwaKuYy9XhszEsfhZOzeKjKcxvG++e+dTAHSUtEOHf4BMA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-docs |
On Fri, Apr 12, 2024 at 1:20 PM Anton Voloshin <a(dot)voloshin(at)postgrespro(dot)ru>
wrote:
> Hello,
>
> While reading "Composite Types" manual page I've noticed that it is
> somewhat hard to follow by trying out given examples. I suggest three
> small changes which would make this page a little easier to follow for
> me:
>
> 1. Clarify "different kind" by adding a link to a section of "create
> type" page
>
> > Note that the AS keyword is essential; without it, the system will
> think a different kind of CREATE TYPE command is meant, and you will get
> odd syntax errors.
>
>
> Here there is no link to CREATE TYPE, so it's not so easy to go there to
> see what is that "different kind" of CREATE TYPE. I suggest to add an
> anchor for the "Base Types" section there and link the words "different
> kind" there.
>
I'd much prefer to leave "different kind" alone and turn the immediately
following, first-on-the-page, instance of CREATE TYPE into a link.
2. make first mention of CREATE TABLE a link
>
> > The syntax is comparable to CREATE TABLE, except ...
>
> It would be useful if this CREATE TABLE (first on this page) would
> become a link.
>
I'm not all that convinced of that particular usefulness but also don't see
it hurting either.
> 3. Simplify CREATE TABLE example to make it self-sufficient
>
> One of the examples on the same "Composite Types" page is an example of
> CREATE TABLE:
>
> CREATE TABLE inventory_item (
> name text,
> supplier_id integer REFERENCES suppliers,
> price numeric CHECK (price > 0)
> );
>
> This example is not self-sufficient: it requires one to have "suppliers"
> table with specific column to work as given.
Agreed.
David J.
From | Date | Subject | |
---|---|---|---|
Next Message | Noah Misch | 2024-04-13 15:00:47 | Re: CREATE ROLE inheritance details |
Previous Message | Anton Voloshin | 2024-04-12 20:20:27 | three small improvements for "Composite Types" page |