| From: | Soumya S Murali <soumyamurali(dot)work(at)gmail(dot)com> |
|---|---|
| To: | Peter Smith <smithpb2250(at)gmail(dot)com> |
| Cc: | Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: Fix how some lists are displayed by psql \d+ |
| Date: | 2026-02-04 05:00:27 |
| Message-ID: | CAMtXxw8mmR4z2kLX0bkOggDQpLrapUkgnM6q9WYYacs=Mg8WGQ@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi all,
Thank you for updating the patches.
On Tue, Feb 3, 2026 at 3:24 PM Peter Smith <smithpb2250(at)gmail(dot)com> wrote:
>
> On Wed, Jan 21, 2026 at 7:30 PM Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com> wrote:
> >
> >
> >
> > > On Jan 21, 2026, at 16:22, Peter Smith <smithpb2250(at)gmail(dot)com> wrote:
> > >
> > > On Wed, Jan 21, 2026 at 7:05 PM Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com> wrote:
> > >>
> > >>
> > >>
> > >>> On Jan 21, 2026, at 10:58, Peter Smith <smithpb2250(at)gmail(dot)com> wrote:
> > >>>
> > >>> A rebase was needed for some more test expected output changes.
> > >>>
> > >>> PSA v2.
> > >>>
> > >>> ======
> > >>> Kind Regards,
> > >>> Peter Smith.
> > >>> Fujitsu Australia
> > >>> <v2-0002-Fix-to-make-psql-d-lists-of-inherits-look-same-as.patch><v2-0001-Fix-to-make-psql-d-lists-of-partitions-look-same-.patch>
> > >>
> > >> I still see some difference:
> > >> ```
> > >> evantest=# \d+ p_test;
> > >> Partitioned table "public.p_test"
> > >> Column | Type | Collation | Nullable | Default | Storage | Compression | Stats target | Description
> > >> ----------+---------+-----------+----------+---------+----------+-------------+--------------+-------------
> > >> id | integer | | | | plain | | |
> > >> username | text | | | | extended | | |
> > >> category | text | | | | extended | | |
> > >> Partition key: LIST (category)
> > >> Indexes:
> > >> "idx_p_test_id" btree (id)
> > >> Partitions:
> > >> p_test_v1 FOR VALUES IN ('A')
> > >> p_test_v2 FOR VALUES IN ('B')
> > >> ```
> > >>
> > >> The index name is quoted, but the partition names are not.
> > >>
> > >
> > > AFAICT
> > > Indexes are quoted
> > > Constraints are quoted
> > > Publications are also quoted
> > >
> > > Partitions are NOT quoted
> > > Inherits are NOT quoted
> > >
> > > ~
> > >
> > > Please confirm --- So, you are expecting that Partitions and Child
> > > tables should also be quoted, so that everything looks the same?
> > >
> > > This is scope creep from the original intent of this thread, so I will
> > > look at doing this in a separate patch 0003.
> > >
> > > ======
> > > Kind Regards
> > > Peter Smith.
> > > Fujitsu Australia
> >
> > Actually I don’t know the rule, which should be quoted and which should not. Is the inconstancy a blocker?
> >
> > So, I was just pointing out the inconsistency, and I want to hear a clarification from someone.
> >
>
> OK. I feel that the "Partitions" and "Child tables" + "Interits" might
> be the odd ones out here, just as those were the odd ones out re the
> list indentation logic.
>
> But I will hold off on adding quotes for them until I hear some
> committer advice that it would be a worthwhile change.
I went through the discussions and verified the unpatched psql display
and after tested the v2 patches on the current master. The patches
applied cleanly, make check and tests passed successfully and using
the patches confirms that "Partitions" and "Child tables" are now
displayed consistently as intended. LGTM from my side.
Regards,
Soumya
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Yugo Nagata | 2026-02-04 05:07:31 | Warn when creating or enabling a subscription with max_logical_replication_workers = 0 |
| Previous Message | Michael Paquier | 2026-02-04 04:58:41 | Re: Add expressions to pg_restore_extended_stats() |