| From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
|---|---|
| To: | Amit Langote <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp> |
| Cc: | Ashutosh Bapat <ashutosh(dot)bapat(at)enterprisedb(dot)com>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Update description of \d[S+] in \? |
| Date: | 2017-08-01 02:06:59 |
| Message-ID: | CA+TgmobKAUxLJ+FX=AmhsP0Jzf4KcPiXKEoot7uap-ZLbD6a=g@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Thu, Jul 13, 2017 at 8:40 PM, Amit Langote
<Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp> wrote:
> On 2017/07/13 19:57, Ashutosh Bapat wrote:
>> On Thu, Jul 13, 2017 at 12:01 PM, Amit Langote
>> <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp> wrote:
>>> The description of \d[S+] currently does not mention that it will list
>>> materialized views and foreign tables. Attached fixes that.
>>>
>>
>> I guess the same change is applicable to the description of \d[S+] NAME as well.
>
> Thanks for the review. Fixed in the attached.
The problem with this, IMV, is that it makes those lines more than 80
characters, whereas right now they are not. And that line seems
doomed to get even longer in the future.
Of course, having it be inaccurate is not great either.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Robert Haas | 2017-08-01 02:10:52 | Re: [PATCH v3] pg_progress() SQL function to monitor progression of long running SQL queries/utilities |
| Previous Message | Robert Haas | 2017-08-01 02:00:53 | Re: Parallel Hash take II |