Re: psql \dh: List High-Level (Root) Tables and Indexes - Mailing list pgsql-hackers

From Greg Sabino Mullane
Subject Re: psql \dh: List High-Level (Root) Tables and Indexes
Date
Msg-id CAKAnmmKuLQ0J4HwT70kTPPUYSGzriuN7oOApky3NQWruW_mPiw@mail.gmail.com
Whole thread Raw
In response to Re: psql \dh: List High-Level (Root) Tables and Indexes  (Sadeq Dousti <msdousti@gmail.com>)
Responses Re: psql \dh: List High-Level (Root) Tables and Indexes
List pgsql-hackers
The patch applies cleanly, and works as advertised. Nice work!

Quick notes:

* doc/src/sgml/ref/psql-ref.sgml

In the varlistentry section, the order should be the same as the other places (N after m)

Line 1644 has an extra comma

Line 1651, maybe the example is simpler as \dNt to keep the wording better, because "indexes that are not partitions" looks odd.

These bits:

  pg_log_error("Did not find any%s relations named \"%s\".",
    no_partition_description, pattern);

are not good for translation. We want things simple with replaceable args/constants, but not replaceable words.

I think the myopt.title ones are fine.

* bin/psql/help.c:

\\dN[Sx+] [PATTERN]     list relation, table, index (no partitions)

better as:

\\dN[Sx+] [PATTERN]     list tables and indexes (no partitions)

Cheers,
Greg

--
Enterprise Postgres Software Products & Tech Support

pgsql-hackers by date:

Previous
From: Sutou Kouhei
Date:
Subject: Re: Make COPY format extendable: Extract COPY TO format implementations
Next
From: Jeff Davis
Date:
Subject: Re: Statistics Import and Export