add tab-complete for memory, serialize option and other minor issues.

From: jian he <jian(dot)universality(at)gmail(dot)com>
To: pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: add tab-complete for memory, serialize option and other minor issues.
Date: 2024-04-27 05:00:00
Message-ID: CACJufxH5UbhbCg-oMt7pHOmvNABF2x48Jfefu24FexSqVgzA3g@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

hi.

I found some minor issues related to the EXPLAIN command.

cannot auto-complete with a white space.
src8=# explain (analyze,b

can auto-complete:
src8=# explain (analyze, b

to make tab-complete work, comma, must be followed with a white space,
not sure why.
--------------
explain (serialize binary) select 1;
ERROR: EXPLAIN option SERIALIZE requires ANALYZE

do you think it's better to rephrase it as:
ERROR: EXPLAIN option SERIALIZE requires ANALYZE option

since we have separate ANALYZE SQL commands.
--------------

<para>
Specify the output format, which can be TEXT, XML, JSON, or YAML.
Non-text output contains the same information as the text output
format, but is easier for programs to parse. This parameter defaults to
<literal>TEXT</literal>.
</para>

should we add <literal> attribute for {TEXT, XML, JSON, YAML} in the
above paragraph?

--------------
i created a patch for tab-complete for memory, SERIALIZE option.

Attachment Content-Type Size
v1-0001-add-Tab-complete-for-EXPLAIN-MEMORY-EXPLAIN-SERIA.patch application/x-patch 1.4 KB

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message John Naylor 2024-04-27 06:14:59 Re: cpluspluscheck/headerscheck require build in REL_16_STABLE
Previous Message Thomas Munro 2024-04-27 04:26:14 Re: WIP: Vectored writeback