Re: psql \dh: List High-Level (Root) Tables and Indexes

2025-07-27 Thread Sadeq Dousti
Dear Matt, Thanks a lot for the nice suggestion. Please find attached the version 7 of the patch, which incorporates your suggestion: It now disregards the N in \dN[ti] if pset.sversion < 10. Best Regards, Sadeq PS: Received an error from the mailer, with the following error message, so remo

Re: psql \dh: List High-Level (Root) Tables and Indexes

2025-04-05 Thread Christoph Berg
Re: Sadeq Dousti > > I think this is the wrong way round. > > It should be \dtN instead of \dNt. > > Hi Christoph, > The order does not matter, the user can use \dNt or \dtN, as they do > exactly the same thing. Letters coming after \d can be freely permuted. If > you mean a change to the document

Re: psql \dh: List High-Level (Root) Tables and Indexes

2025-03-28 Thread Sadeq Dousti
The S flag is "universal" enough and can be combined with a wide variety of commands to show system views. Examples include \d, \dn, \dp, \dL, and so on. In contrast, the N I'm introducing can only be combined with \d, so I wouldn't treat it the same as S. All things that combine with \d (t, i, m,

Re: psql \dh: List High-Level (Root) Tables and Indexes

2025-03-27 Thread Sadeq Dousti
Sorry Greg, Just understood what you mean. Please find attached the v6 of the patch. Best regards, Sadeq v6-0001-psql-acommand-for-non-partitioned-tables-indexes.patch Description: Binary data

Re: psql \dh: List High-Level (Root) Tables and Indexes

2025-03-27 Thread Sadeq Dousti
> I think this is the wrong way round. > It should be \dtN instead of \dNt. Hi Christoph, The order does not matter, the user can use \dNt or \dtN, as they do exactly the same thing. Letters coming after \d can be freely permuted. If you mean a change to the documentation or tests, I can apply wha

Re: psql \dh: List High-Level (Root) Tables and Indexes

2025-03-27 Thread Sadeq Dousti
> I think it's fine the way it is, with regards to v10 check. Can you post a > rebased patch? > Hi Greg, I just checked here: https://commitfest.postgresql.org/patch/5594/ Seems the patch is OK with the latest master, and no rebase is needed. Do you mean that instead of a diff, I post a patch? B

Re: psql \dh: List High-Level (Root) Tables and Indexes

2025-03-27 Thread Christoph Berg
Re: Sadeq Dousti > Thanks Greg and others for the feedback! > > Please find attached the patch for implementing \dN (including \dNt, \dNi, > \dNit). I don't care particularly about the choice of letter, but I think this is the wrong way round. It should be \dtN instead of \dNt. Is the form \dN a

Re: psql \dh: List High-Level (Root) Tables and Indexes

2025-03-27 Thread Greg Sabino Mullane
I think it's fine the way it is, with regards to v10 check. Can you post a rebased patch? Cheers, Greg -- Crunchy Data - https://www.crunchydata.com Enterprise Postgres Software Products & Tech Support

Re: psql \dh: List High-Level (Root) Tables and Indexes

2025-02-27 Thread Sadeq Dousti
Thanks Vignesh for the review! > Currently we are supporting only PG13 and higher versions. I understand that servers older than PG13 are no longer supported. But on the client side, we still have this notice at the top of describe.c file, which indicates that the client should support 9.2+. *

Re: psql \dh: List High-Level (Root) Tables and Indexes

2025-02-26 Thread vignesh C
On Wed, 26 Feb 2025 at 23:21, Sadeq Dousti wrote: > > Thanks a lot Greg! > > > Changes look good to me, thanks. Can you make a new patch that applies a > > single set of changes to HEAD? > > Please find attached the diff to Head in a single file. Currently we are supporting only PG13 and higher

Re: psql \dh: List High-Level (Root) Tables and Indexes

2025-02-26 Thread Sadeq Dousti
Thanks a lot Greg! > Changes look good to me, thanks. Can you make a new patch that applies a single set of changes to HEAD? Please find attached the diff to Head in a single file. Best Regards, Sadeq Dousti diff --git a/doc/src/sgml/ref/psql-ref.sgml b/doc/src/sgml/ref/psql-ref.sgml index cedcc

Re: psql \dh: List High-Level (Root) Tables and Indexes

2025-02-26 Thread Greg Sabino Mullane
Changes look good to me, thanks. Can you make a new patch that applies a single set of changes to HEAD? Cheers, Greg -- Crunchy Data - https://www.crunchydata.com Enterprise Postgres Software Products & Tech Support

Re: psql \dh: List High-Level (Root) Tables and Indexes

2025-02-25 Thread Sadeq Dousti
Dear Greg, Thank you so much for the kind and prompt review! Please find the patches attached. The second patch (0002) is where I applied the requested changes. Best regards, Sadeq Dousti On Wed, Feb 26, 2025 at 1:01 AM Greg Sabino Mullane wrote: > The patch applies cleanly, and works as adv

Re: psql \dh: List High-Level (Root) Tables and Indexes

2025-02-25 Thread Greg Sabino Mullane
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 wordin

Re: psql \dh: List High-Level (Root) Tables and Indexes

2025-02-25 Thread Sadeq Dousti
Thanks Greg and others for the feedback! Please find attached the patch for implementing \dN (including \dNt, \dNi, \dNit). Best Regards, Sadeq Dousti From d8fde4b05eee95089548384c07b59304f2fecc1c Mon Sep 17 00:00:00 2001 From: Sadeq Dousti <3616518+msdou...@users.noreply.github.com> Date: Tue, 2

Re: psql \dh: List High-Level (Root) Tables and Indexes

2025-02-24 Thread Greg Sabino Mullane
On Mon, Feb 24, 2025 at 3:07 AM Sadeq Dousti wrote: > * While capitalization of N might seem arbitrary, it can be acceptable as > it's "negating" some concept > ... > > If there are no hard objections, I'll proceed with adding the N letter to > \d commands. > Not strongly opposed, but "N" is a

Re: psql \dh: List High-Level (Root) Tables and Indexes

2025-02-24 Thread Sadeq Dousti
Dear all, I'm grateful for taking your time and providing various solutions! Personally, I like Tom's idea of using N better: * It can stand for "not a partition member" or "no partitions", and does not provide any new terminology. * It can be combined with various commands \dN, \dtN (or \dNt),

Re: psql \dh: List High-Level (Root) Tables and Indexes

2025-02-23 Thread Pavel Stehule
po 24. 2. 2025 v 6:16 odesílatel Tom Lane napsal: > "David G. Johnston" writes: > > How about: > > \pset partition_roots_only 'true' > > FWIW, I don't especially love this idea. None of our existing > \d-type commands use pset parameters for filtering, and I don't > see a reason to believe that

Re: psql \dh: List High-Level (Root) Tables and Indexes

2025-02-23 Thread Tom Lane
"David G. Johnston" writes: > How about: > \pset partition_roots_only 'true' FWIW, I don't especially love this idea. None of our existing \d-type commands use pset parameters for filtering, and I don't see a reason to believe that people would typically want this to be a persistent setting. I'

Re: psql \dh: List High-Level (Root) Tables and Indexes

2025-02-23 Thread David G. Johnston
On Sun, Feb 23, 2025 at 4:31 PM Greg Sabino Mullane wrote: > On Sun, Feb 23, 2025 at 4:34 PM Sadeq Dousti wrote: > >> Totally agree about the naming. There's no other terminology known to me, >> but I'll definitely think about it. Of course, I'm very open to >> suggestions from you or anyone els

Re: psql \dh: List High-Level (Root) Tables and Indexes

2025-02-23 Thread Greg Sabino Mullane
On Sun, Feb 23, 2025 at 4:34 PM Sadeq Dousti wrote: > Totally agree about the naming. There's no other terminology known to me, > but I'll definitely think about it. Of course, I'm very open to > suggestions from you or anyone else in the community. > The problem is that we are really tight on a

Re: psql \dh: List High-Level (Root) Tables and Indexes

2025-02-23 Thread Tom Lane
Sadeq Dousti writes: >> "High-level" is not terminology we use anywhere else >> We do use the terms "root partition" or "partition root". > I agree "high-level" is not standard, but "root partition" also entails > the table is actually partitioned. This is not necessarily the case. > The suggeste

Re: psql \dh: List High-Level (Root) Tables and Indexes

2025-02-23 Thread Sadeq Dousti
> That definition seems bizarre, and even less derivable > from the term "high-level" than what I thought you meant. > You really need some other terminology, I think. Totally agree about the naming. There's no other terminology known to me, but I'll definitely think about it. Of course, I'm very

Re: psql \dh: List High-Level (Root) Tables and Indexes

2025-02-23 Thread Sadeq Dousti
Hi Tom, Thanks a lot for looking into this! > "High-level" is not terminology we use anywhere else > We do use the terms "root partition" or "partition root". I agree "high-level" is not standard, but "root partition" also entails the table is actually partitioned. This is not necessarily the ca

Re: psql \dh: List High-Level (Root) Tables and Indexes

2025-02-23 Thread Sadeq Dousti
> Sorry an id was missing in psql documentation, thus CI was failing. Combining patches into one, as otherwise CI does not combine them. Best Regards Sadeq Dousti From ad81b71fe67174aebd7f6c02a82aa0ab88ff Mon Sep 17 00:00:00 2001 From: Sadeq Dousti <3616518+msdousti@users.noreply.github.com>

Re: psql \dh: List High-Level (Root) Tables and Indexes

2025-02-23 Thread Tom Lane
Sadeq Dousti writes: > Please find attached the patch to add a new metacommand (\dh) to psql. I don't have a strong feeling one way or the other about whether this is useful functionality or not. But if we do want it, I'd like to bikeshed on the name a bit. "High-level" is not terminology we us

Re: psql \dh: List High-Level (Root) Tables and Indexes

2025-02-23 Thread Sadeq Dousti
Sorry an id was missing in psql documentation, thus CI was failing. Please find attached a patch that fixes this issue. Best Regards, Sadeq Dousti Tech Lead and Staff Software Engineer at Trade Republic Bank GmbH From c7b7f39c171648f3193cb9d61a4815ac32b779e3 Mon Sep 17 00:00:00 2001 From: Sadeq <3