Re: PATCH: Unlogged tables re-initialization tests - Mailing list pgsql-hackers

From Thomas Munro
Subject Re: PATCH: Unlogged tables re-initialization tests
Date
Msg-id CAEepm=1o=Dqvhad1KX6b==O3Z3jU4X3ZO+fOYbxN_s2WW4G7WA@mail.gmail.com
Whole thread Raw
In response to PATCH: Unlogged tables re-initialization tests  (David Steele <david@pgmasters.net>)
Responses Re: PATCH: Unlogged tables re-initialization tests  (David Steele <david@pgmasters.net>)
List pgsql-hackers
On Thu, Mar 1, 2018 at 9:24 AM, David Steele <david@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


pgsql-hackers by date:

Previous
From: Andres Freund
Date:
Subject: Re: Why chain of snapshots is used in ReorderBufferCommit?
Next
From: Arseny Sher
Date:
Subject: Two-phase update of restart_lsn in LogicalConfirmReceivedLocation