On Sun, Mar 27, 2022 at 11:07:27AM -0500, Jaime Casanova wrote:
> > > It lacks documentation and I need help with WIN32 part of it, I created
> > > this new mail to put the patch on the next commitfest.
> >
> > The patch currently fails on cfbot due to warnings, likely related due to the
> > win32 issue: https://cirrus-ci.com/task/4566046517493760?logs=mingw_cross_warning#L388
> >
> > As it's a new patch submitted to the last CF, hasn't gotten any review yet and
> > misses some platform support, it seems like there's no chance it can make it
> > into 15?
>
> Because I have zero experience on the windows side of this, I will take
> some time to complete that part.
>
> Should we move this to the next commitfest (and make 16 the target for
> this)?
Done.
src/tools/ci/README may help test this under windows, but that's probably not enough
to allow writing the win-specific parts.
I guess you'll need to write tests for this..unfortunately that requires files
>1GB in size, unless you recompile postgres :(
It may be good enough to write an 0002 patch meant for CI only, but not
intended to be merged. That can create a 2300MB table in src/test/regress, and
change pg_upgrade to run with (or default to) multiple jobs per tablespace.
Make sure it fails if the loop around relfilenodes doesn't work.
I can't help with win32, but that would be enough to verify it if someone else
fills in the windows parts.
--
Justin
