Request a Demo
Home   >   mailing lists

Re: Psql meta-command conninfo+ - Mailing list pgsql-hackers

From Hunaid Sohail
Subject Re: Psql meta-command conninfo+
Date
Msg-id CAMWA6ybKwxL6ez7++CO9n7huUvyaT1jo4VEtidNtX4KgQu04BA@mail.gmail.com
Whole thread Raw
In response to Re: Psql meta-command conninfo+  (Alvaro Herrera <alvherre@alvh.no-ip.org>)
Responses Re: Psql meta-command conninfo+
List pgsql-hackers
Tree view
Hi,

On Thu, Jan 16, 2025 at 6:01 PM Alvaro Herrera <alvherre@alvh.no-ip.org> wrote:
On 2025-Jan-16, Hunaid Sohail wrote:

>  server_encoding               | UTF8
>  server_version                | 18devel
>  client_encoding               | UTF8
>  session_authorization         | hunaid
>  standard_conforming_strings   | on
>  DateStyle                     | ISO, MDY
>  scram_iterations              | 4096
>  default_transaction_read_only | off
>  application_name              | psql
>  is_superuser                  | on
>  search_path                   | "$user", public
>  IntervalStyle                 | postgres
>  TimeZone                      | Asia/Karachi
>  integer_datetimes             | on
>  in_hot_standby                | off

Wait a second, why do we have these here?  Aren't they already in
\dconfig?

There are indeed some overlaps because Sami [1] and David [2] suggested
to show all parameters, with Sami specifically asking to include in_hot_standby
and application_name, as they can be useful. Moreover, parameters like
server_encoding and session_authorization were already included in previous patches.

In other cases, we'd have to pick and choose which parameters to include.
If a new parameter is reported that might be relevant to this meta command and
the user wants it included, we would need to modify the code repeatedly.

By the way, I just noticed CF bot failed with the error:
'undefined reference to palloc'.
Maybe I should use malloc as it is used in the file, and free with PQmemfree().


Regards,
Hunaid Sohail

pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: Automatic update of time column
Next
From: Tom Lane
Date:
Subject: Re: leafhopper: Assert("outerstartsel <= outerendsel"), File: "costsize.c"