Horton

Thread

Commits

  1. Add psql tab compression for ALTER TABLE .. { OF | NOT OF }

  2. Add psql tab compression for SET COMPRESSION with ALTER TABLE

  1. [PATCH] Tab completion for SET COMPRESSION

    Aleksander Alekseev <aleksander@timescale.com> — 2022-08-22T12:48:56Z

    Hi hackers,

The proposed patch adds the missing tab completion for 'ALTER TABLE
... SET COMPRESSION ...' syntax.

-- 
Best regards,
Aleksander Alekseev

  2. Re: [PATCH] Tab completion for SET COMPRESSION

    Shinya Kato <shinya11.kato@oss.nttdata.com> — 2022-09-06T00:54:53Z

    On 2022-08-22 21:48, Aleksander Alekseev wrote:
> Hi hackers,
> 
> The proposed patch adds the missing tab completion for 'ALTER TABLE
> ... SET COMPRESSION ...' syntax.
Thanks, LGTM.

In addition, why not take this opportunity to create a tab completion 
for "ALTER TABLE <name> OF <type_name>" and "ALTER TABLE <name> NOT OF"?


-- 
Regards,

--
Shinya Kato
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION

  3. Re: [PATCH] Tab completion for SET COMPRESSION

    Michael Paquier <michael@paquier.xyz> — 2022-09-06T06:41:41Z

    On Tue, Sep 06, 2022 at 09:54:53AM +0900, Shinya Kato wrote:
> In addition, why not take this opportunity to create a tab completion for
> "ALTER TABLE <name> OF <type_name>" and "ALTER TABLE <name> NOT OF"?

Right.  That looks fine to me, so applied.
--
Michael

  4. Re: [PATCH] Tab completion for SET COMPRESSION

    Aleksander Alekseev <aleksander@timescale.com> — 2022-09-06T08:28:10Z

    Hi hackers,

> Right.  That looks fine to me, so applied.

Thanks, Michael.

> In addition, why not take this opportunity to create a tab completion for
> "ALTER TABLE <name> OF <type_name>" and "ALTER TABLE <name> NOT OF"?

Thanks for reviewing, Shinya. Let's fix this too. The patch is attached.

-- 
Best regards,
Aleksander Alekseev

  5. Re: [PATCH] Tab completion for SET COMPRESSION

    Shinya Kato <shinya11.kato@oss.nttdata.com> — 2022-09-06T10:32:20Z

    On 2022-09-06 17:28, Aleksander Alekseev wrote:

>> In addition, why not take this opportunity to create a tab completion 
>> for
>> "ALTER TABLE <name> OF <type_name>" and "ALTER TABLE <name> NOT OF"?
> 
> Thanks for reviewing, Shinya. Let's fix this too. The patch is 
> attached.

Thanks for the new patch!
A minor modification has been made so that the composite type is also 
completed after "ALTER TABLE <name> OF".

Thought?

-- 
Regards,

--
Shinya Kato
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION

  6. Re: [PATCH] Tab completion for SET COMPRESSION

    Aleksander Alekseev <aleksander@timescale.com> — 2022-09-06T11:57:55Z

    Hi Shinya,

> A minor modification has been made so that the composite type is also
> completed after "ALTER TABLE <name> OF".

LGTM. Here is v3 created with `git format-path`. Unlike v2 it also
includes the commit message.

-- 
Best regards,
Aleksander Alekseev

  7. Re: [PATCH] Tab completion for SET COMPRESSION

    Shinya Kato <shinya11.kato@oss.nttdata.com> — 2022-09-08T07:40:32Z

    On 2022-09-06 20:57, Aleksander Alekseev wrote:
> Hi Shinya,
> 
>> A minor modification has been made so that the composite type is also
>> completed after "ALTER TABLE <name> OF".
> 
> LGTM. Here is v3 created with `git format-path`. Unlike v2 it also
> includes the commit message.

Thanks! I marked it as ready for committer.

-- 
Regards,

--
Shinya Kato
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION

  8. Re: [PATCH] Tab completion for SET COMPRESSION

    Michael Paquier <michael@paquier.xyz> — 2022-09-10T08:23:26Z

    On Thu, Sep 08, 2022 at 04:40:32PM +0900, Shinya Kato wrote:
> Thanks! I marked it as ready for committer.

I thought that there was a gotcha in this area for composite types,
but on second look it looks that I was wrong.  Hence, applied.
--
Michael