Re: Refactor ReindexStmt and its "concurrent" boolean

2020-09-03 Thread Michael Paquier
On Thu, Sep 03, 2020 at 11:14:38AM +0900, Michael Paquier wrote: > It should, thanks for looking at it. Let's wait a couple of days and > see if others have any comments. If there are no objections, I'll try > to commit this one. And applied. -- Michael signature.asc Description: PGP signature

Re: Refactor ReindexStmt and its "concurrent" boolean

2020-09-02 Thread Michael Paquier
On Wed, Sep 02, 2020 at 01:17:32PM +0200, Julien Rouhaud wrote: > Shouldn't options be an int? The rest of the patch looks good to me. It should, thanks for looking at it. Let's wait a couple of days and see if others have any comments. If there are no objections, I'll try to commit this one. -

Re: Refactor ReindexStmt and its "concurrent" boolean

2020-09-02 Thread Julien Rouhaud
On Wed, Sep 2, 2020 at 1:03 PM Michael Paquier wrote: > > Hi all, > > $subject has been mentioned a couple of times, including today: > https://www.postgresql.org/message-id/20200902010012.ge1...@paquier.xyz > > We have a boolean argument in ReindexStmt to control a concurrent > run, and we also h

Refactor ReindexStmt and its "concurrent" boolean

2020-09-02 Thread Michael Paquier
Hi all, $subject has been mentioned a couple of times, including today: https://www.postgresql.org/message-id/20200902010012.ge1...@paquier.xyz We have a boolean argument in ReindexStmt to control a concurrent run, and we also have in parallel of that a bitmask to control the options of the state