Re: PATCH: Unlogged tables re-initialization tests

Поиск
Список
Период
Сортировка
От Thomas Munro
Тема Re: PATCH: Unlogged tables re-initialization tests
Дата
Msg-id CAEepm=1o=Dqvhad1KX6b==O3Z3jU4X3ZO+fOYbxN_s2WW4G7WA@mail.gmail.com
обсуждение исходный текст
Ответ на PATCH: Unlogged tables re-initialization tests  (David Steele <david@pgmasters.net>)
Ответы Re: PATCH: Unlogged tables re-initialization tests  (David Steele <david@pgmasters.net>)
Список 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 по дате отправления:

Предыдущее
От: Andres Freund
Дата:
Сообщение: Re: Why chain of snapshots is used in ReorderBufferCommit?
Следующее
От: Arseny Sher
Дата:
Сообщение: Two-phase update of restart_lsn in LogicalConfirmReceivedLocation