Re: should CREATE INDEX ON partitioned_table callPreventInTransactionBlock() ? - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: should CREATE INDEX ON partitioned_table callPreventInTransactionBlock() ?
Date
Msg-id 20200608152726.GA7241@alvherre.pgsql
Whole thread Raw
In response to should CREATE INDEX ON partitioned_table callPreventInTransactionBlock() ?  (Justin Pryzby <pryzby@telsasoft.com>)
Responses Re: should CREATE INDEX ON partitioned_table callPreventInTransactionBlock() ?
List pgsql-hackers
On 2020-Jun-08, Justin Pryzby wrote:

> This blocks writes to all partitions until commit:
> 
> postgres=# begin; CREATE INDEX ON pt(i);
> BEGIN
> CREATE INDEX
> 
> Compare with CLUSTER rel1, rel2, ..., and REINDEX {SCHEMA|DATABASE|SYSTEM},
> which release their locks as soon as each rel is processed.

Well, that would also require that transactions are committed and
started for each partition.  Merely adding PreventInTransactionBlock
would not do that.  Moreover, since this would break DDL-in-transactions
that would otherwise work, it should be optional and thus need a keyword
in the command.  But CONCURRENTLY isn't it (because that means something
else) so we'd have to discuss what it would be.

> I noticed while implementing REINDEX for partitioned tables, which, it seems
> clear, should also avoid slowly accumulating more and more write locks across
> an entire partition heirarchy.

Right.

-- 
Álvaro Herrera                https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services



pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: snowball release
Next
From: Justin Pryzby
Date:
Subject: Re: should CREATE INDEX ON partitioned_table callPreventInTransactionBlock() ?