This patch also applied with minor offsets, compiled without
warning, and passes regression tests, including `make check-world`
with TAP tests enabled. It looks good to me, presenting a cleaner
API.
I really want to get this to the point where we can dump and
restore nested materialized views bet
I wrote:
> In the attached patch for this, I took a middle ground of separating out
> the command and status string building and parsing functions. There isn't
> presently any provision for letting archive format modules override these,
> but that could easily be added if we ever need it. Meanwhi
I wrote:
> I am pretty strongly tempted to get rid of MasterStartParallelItem
> altogether and just hard-code what it does in DispatchJobForTocEntry.
> ...
> A different line of thought would be to fix the worker-command-parsing
> situation by allowing the parsing to happen in format-specific code,
pg_dump's APIs for parallel dump/restore include an
archive-format-specific MasterStartParallelItem function, which is
evidently intended to allow format-specific data to be sent to the
worker process during startup of a parallel sub-job. However, no
such data can be sent in practice, because pars