pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Alvaro Herrera <alvherre@alvh.no-ip.org>
To: Alexander Korotkov <akorotkov@postgresql.org>
Cc: pgsql-committers@lists.postgresql.org
Subject: Re: pgsql: Adjust parallel_schedule with event triggers on authenticated lo
Date: Fri, 27 Oct 2023 12:47:44 +0200
Message-ID: <202310271047.mnwkql6nhbwi@alvherre.pgsql> (raw)
In-Reply-To: <E1qwIIq-002YkM-PJ@gemulon.postgresql.org>

On 2023-Oct-27, Alexander Korotkov wrote:

> Adjust parallel_schedule with event triggers on authenticated login
> 
> Event triggers on authenticated login could catch the connection of the
> concurrent test.  In order to prevent this side effect we need to run
> the event_trigger test alone.

Hmm, not a fan, because the original lines of the event_trigger test,
which had no problem running in parallel, no longer do so.  Why not
split out only the new part of the test to run in non-parallel mode?
An effortless win of ~50 ms.

After this patch:

ok 211       + stats                                     644 ms
# parallel group (2 tests):  event_trigger oidjoins
ok 212       + oidjoins                                  103 ms
ok 213       + event_trigger                              57 ms
ok 214       - event_trigger_login                        16 ms
ok 215       - fast_default                               66 ms
ok 216       - tablespace                                156 ms
1..216
# All 216 tests passed.


Before:

ok 211       + stats                                     603 ms
ok 212       - event_trigger                              66 ms
ok 213       - oidjoins                                  113 ms
ok 214       - fast_default                               71 ms
ok 215       - tablespace                                161 ms
1..215
# All 215 tests passed.

-- 
Álvaro Herrera        Breisgau, Deutschland  —  https://www.EnterpriseDB.com/

Attachments:

  [text/x-diff] 0001-paralelize-stuff.patch (0B, ../202310271047.mnwkql6nhbwi@alvherre.pgsql/2-0001-paralelize-stuff.patch)
  download

view thread (5+ messages)  latest in thread

Message-ID: <202310271047.mnwkql6nhbwi@alvherre.pgsql>
Permalink:  ../202310271047.mnwkql6nhbwi@alvherre.pgsql/
Also on:    postgresql.org/message-id/202310271047.mnwkql6nhbwi@alvherre.pgsql

 ·  · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-committers@postgresql.org
  Cc: alvherre@alvh.no-ip.org, akorotkov@postgresql.org, pgsql-committers@lists.postgresql.org
  Subject: Re: pgsql: Adjust parallel_schedule with event triggers on authenticated lo
  In-Reply-To: <202310271047.mnwkql6nhbwi@alvherre.pgsql>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox