From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> |
Cc: | Magnus Hagander <magnus(at)hagander(dot)net>, Isaac Morland <isaac(dot)morland(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Remove source code display from \df+? |
Date: | 2023-01-11 21:11:46 |
Message-ID: | 669149.1673471506@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> writes:
> st 11. 1. 2023 v 19:31 odesílatel Magnus Hagander <magnus(at)hagander(dot)net>
> napsal:
>> This is only about Internal and C, isn't it? Isn't the oid of these
>> static, and identified by INTERNALlanguageId and ClanguageId respectively?
>> So we could just have the query show the prosrc column if the language oid
>> is one of those two, and otherwise show "Please use \sf to view the
>> source"?
> I think it can be acceptable - maybe we can rename the column "source code"
> like "internal name" or some like that.
Yeah, "source code" has always been kind of a misnomer for these
languages.
> again I don't think printing "Please use \sf to view the source"? " often
> can be user friendly. \? is clear and \sf is easy to use
We could shorten it to "See \sf" or something like that. But if we change
the column header to "internal name" or the like, then the column just
obviously doesn't apply for non-internal languages, so leaving it null
should be fine.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2023-01-11 21:13:01 | Re: doc: mentioned CREATE+ATTACH PARTITION as an alternative to CREATE TABLE..PARTITION OF |
Previous Message | Andres Freund | 2023-01-11 21:09:42 | Re: Option to not use ringbuffer in VACUUM, using it in failsafe mode |