On Mon, Mar 6, 2017 at 11:05 AM, Simon Riggs <simon@2ndquadrant.com> wrote:
> On 6 March 2017 at 05:29, Ashutosh Bapat
> <ashutosh.bapat@enterprisedb.com> wrote:
>
>> Just to confirm, you want the output to look like this
>>>> \d+ t1
>>>> Table "public.t1"
>>>> Column | Type | Collation | Nullable | Default | Storage | Stats
>>>> target | Description
>>>> --------+---------+-----------+----------+---------+---------+--------------+-------------
>>>> a | integer | | not null | | plain | |
>>>> Partition key: RANGE (a)
>>>> Partitions: t1p1 FOR VALUES FROM (0) TO (100), HAS PARTITIONS
>>>> t1p2 FOR VALUES FROM (100) TO (200)
>>
>>>
>>> lowercase please
>>
>> Except for HAS PARTITIONS, everything is part of today's output. Given
>> the current output, HAS PARTITIONS should be in upper case.
>
> "has partitions" is not part of the DDL, whereas "FOR VALUES FROM (0)
> TO (100)" is. So ISTM sensible to differentiate between DDL and
> non-ddl using upper and lower case.
>
Make sense. Will try to cook up a patch soon.
--
Best Wishes,
Ashutosh Bapat
EnterpriseDB Corporation
The Postgres Database Company