On Thu, Mar 1, 2018 at 9:24 AM, David Steele <da...@pgmasters.net> wrote: > These tests were originally included in the exclude unlogged tables > patch [1] to provide coverage for the refactoring of reinit.c.
Hi David, +# The following tests test symlinks. Windows doesn't have symlinks, so +# skip on Windows. Could you please explain this a bit more? I don't see any symlinks being used directly in this test. I do see CREATE TABLESPACE and that uses symlink(), but win32_port.h converts that to "junctions", apparently the Windows equivalent. Is there some reason that doesn't work with this test? If, by any chance, you are talking about whatever dark force prevents the "tablespace" regression test from succeeding when run as a user with administrative privileges on Windows, then I would *love* to hear an explanation for that phenomenon and how to fix it, because it currently prevents me from automatically testing all Commitfest patches on the appveyor.com Windows build farm. I know that it passes as a non-administrative user. -- Thomas Munro http://www.enterprisedb.com