|From:||Yugo NAGATA <nagata(at)sraoss(dot)co(dot)jp>|
|To:||Michael Paquier <michael(at)paquier(dot)xyz>|
|Subject:||Re: Tab completion for ALTER MATERIALIZED VIEW ... SET ACCESS METHOD|
|Views:||Raw Message | Whole Thread | Download mbox | Resend email|
On Sat, 19 Mar 2022 19:31:59 +0900
Michael Paquier <michael(at)paquier(dot)xyz> wrote:
> On Fri, Mar 18, 2022 at 03:13:05PM +0900, Michael Paquier wrote:
> > Thanks. This looks rather sane to me. I'll split things into 3
> > commits in total, as of the psql completion, SET TABLESPACE and SET
> > ACCESS METHOD. The first and third patches are only for HEAD, while
> > the documentation hole with SET TABLESPACE should go down to v10.
> > Backpatching the tests of ALTER MATERIALIZED VIEW ALL IN TABLESPACE
> > would not hurt, either, as there is zero coverage for that now.
> I have applied the set, after splitting things mostly as mentioned
> - The doc change for SET TABLESPACE on ALTER MATVIEW has been
> - The regression tests for SET TABLESPACE have been applied on HEAD,
> as these are independent of the rest, good on their own.
> - All the remaining parts for SET ACCESS METHOD (psql tab completion,
> tests and docs) have been merged together on HEAD. I could not
> understand why the completion done after SET ACCESS METHOD was not
> grouped with the other parts for ALTER MATVIEW, so I have moved the
> new entry there, and I have added a test checking after an error for
> multiple subcommands, while on it.
> Thanks, Nagata-san!
Yugo NAGATA <nagata(at)sraoss(dot)co(dot)jp>
|Next Message||Dilip Kumar||2022-03-22 09:00:05||Re: [Proposal] Fully WAL logged CREATE DATABASE - No Checkpoints|
|Previous Message||Daniel Gustafsson||2022-03-22 08:43:51||Re: On login trigger: take three|