Re: pgsql: Harden new test case against force_parallel_mode = regress.

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Robert Haas <robertmhaas(at)gmail(dot)com>
Cc: "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: pgsql: Harden new test case against force_parallel_mode = regress.
Date: 2023-03-03 16:37:56
Message-ID: 2478846.1677861476@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

Robert Haas <robertmhaas(at)gmail(dot)com> writes:
> On Thu, Mar 2, 2023 at 5:47 PM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>> Per buildfarm: worker processes can't see a role created in
>> the current transaction.

> Now why would that happen? Surely the snapshot for each command is
> passed down from leader to worker, and the worker is not free to
> invent a snapshot from nothing.

The workers were failing at startup, eg (from [1]):

+ERROR: role "regress_psql_user" does not exist
+CONTEXT: while setting parameter "session_authorization" to "regress_psql_user"

Maybe this says that worker startup needs to install the snapshot before
doing any catalog accesses? Anyway, I'd be happy to revert this test
hack if you care to make the case work.

regards, tom lane

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Matthias van de Meent 2023-03-03 16:38:08 Re: pgsql: Harden new test case against force_parallel_mode = regress.
Previous Message Robert Haas 2023-03-03 16:16:28 Re: pgsql: Harden new test case against force_parallel_mode = regress.

Browse pgsql-hackers by date

  From Date Subject
Next Message Matthias van de Meent 2023-03-03 16:38:08 Re: pgsql: Harden new test case against force_parallel_mode = regress.
Previous Message Tom Lane 2023-03-03 16:33:29 Re: libpq: PQgetCopyData() and allocation overhead