On Sat, Dec 12, 2020 at 01:45:26PM -0600, Justin Pryzby wrote:
> It's a lot like what I wrote as [PATCH v31 1/5] ExecReindex and ReindexParams
> In my v31 patch, I moved ReindexOptions to a private structure in indexcmds.c,
> with an "int options" bitmask which is passed to reindex_index() et al.  Your
> patch keeps/puts ReindexOptions index.h, so it also applies to reindex_index,
> which I think is good.

a3dc926 is an equivalent of 0001~0003 merged together.  0008 had
better be submitted into a separate thread if there is value to it.
0004~0007 are the pieces remaining.  Could it be possible to rebase
things on HEAD and put the tablespace bits into the structures 
{Vacuum,Reindex,Cluster}Params?
--
Michael

Attachment: signature.asc
Description: PGP signature

Reply via email to