pg.ddx.io pgsql-committers@postgresql.org mailing list archive
help / color / mirror / Atom feedpgsql: Adjust parallel_schedule with event triggers on authenticated lo
5+ messages / 3 participants
[nested] [flat]
* pgsql: Adjust parallel_schedule with event triggers on authenticated lo
@ 2023-10-27 08:34 Alexander Korotkov <akorotkov@postgresql.org>
0 siblings, 1 reply; 5+ messages in thread
From: Alexander Korotkov @ 2023-10-27 08:34 UTC (permalink / raw)
To: pgsql-committers@lists.postgresql.org
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.
Discussion: https://postgr.es/m/flat/CAMEv5_sS5G7K1PCV2oBx5+1NF1ZneJ6C5Z_xV_SWsZMukdFZiA@mail.gmail.com
Author: Mikhail A. Gribkov
Reviewed-by: Aleksander Alekseev
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/83510534d5f3f116efa035639b9b62b8c6c4df34
Modified Files
--------------
src/test/regress/parallel_schedule | 10 ++++++----
1 file changed, 6 insertions(+), 4 deletions(-)
^ permalink raw reply [nested|flat] 5+ messages in thread
* Re: pgsql: Adjust parallel_schedule with event triggers on authenticated lo
@ 2023-10-27 10:47 Alvaro Herrera <alvherre@alvh.no-ip.org>
parent: Alexander Korotkov <akorotkov@postgresql.org>
0 siblings, 1 reply; 5+ messages in thread
From: Alvaro Herrera @ 2023-10-27 10:47 UTC (permalink / raw)
To: Alexander Korotkov <akorotkov@postgresql.org>; +Cc: pgsql-committers@lists.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
^ permalink raw reply [nested|flat] 5+ messages in thread
* Re: pgsql: Adjust parallel_schedule with event triggers on authenticated lo
@ 2023-10-27 11:20 Alexander Korotkov <aekorotkov@gmail.com>
parent: Alvaro Herrera <alvherre@alvh.no-ip.org>
0 siblings, 1 reply; 5+ messages in thread
From: Alexander Korotkov @ 2023-10-27 11:20 UTC (permalink / raw)
To: Alvaro Herrera <alvherre@alvh.no-ip.org>; +Cc: Alexander Korotkov <akorotkov@postgresql.org>; pgsql-committers@lists.postgresql.org
On Fri, Oct 27, 2023 at 1:48 PM Alvaro Herrera <alvherre@alvh.no-ip.org> wrote:
> 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.
OK, I'm good with this.
Do you like me to push this? Or do you like to push it yourself?
------
Regards,
Alexander Korotkov
^ permalink raw reply [nested|flat] 5+ messages in thread
* Re: pgsql: Adjust parallel_schedule with event triggers on authenticated lo
@ 2023-10-27 11:51 Alvaro Herrera <alvherre@alvh.no-ip.org>
parent: Alexander Korotkov <aekorotkov@gmail.com>
0 siblings, 1 reply; 5+ messages in thread
From: Alvaro Herrera @ 2023-10-27 11:51 UTC (permalink / raw)
To: Alexander Korotkov <aekorotkov@gmail.com>; +Cc: Alexander Korotkov <akorotkov@postgresql.org>; pgsql-committers@lists.postgresql.org
On 2023-Oct-27, Alexander Korotkov wrote:
> OK, I'm good with this.
> Do you like me to push this? Or do you like to push it yourself?
Please do.
--
Álvaro Herrera Breisgau, Deutschland — https://www.EnterpriseDB.com/
^ permalink raw reply [nested|flat] 5+ messages in thread
* Re: pgsql: Adjust parallel_schedule with event triggers on authenticated lo
@ 2023-10-27 20:53 Alexander Korotkov <aekorotkov@gmail.com>
parent: Alvaro Herrera <alvherre@alvh.no-ip.org>
0 siblings, 0 replies; 5+ messages in thread
From: Alexander Korotkov @ 2023-10-27 20:53 UTC (permalink / raw)
To: Alvaro Herrera <alvherre@alvh.no-ip.org>; +Cc: Alexander Korotkov <akorotkov@postgresql.org>; pgsql-committers@lists.postgresql.org
On Fri, Oct 27, 2023 at 2:51 PM Alvaro Herrera <alvherre@alvh.no-ip.org> wrote:
> On 2023-Oct-27, Alexander Korotkov wrote:
>
> > OK, I'm good with this.
> > Do you like me to push this? Or do you like to push it yourself?
>
> Please do.
Done, thanks.
------
Regards,
Alexander Korotkov
^ permalink raw reply [nested|flat] 5+ messages in thread
end of thread, other threads:[~2023-10-27 20:53 UTC | newest]
Thread overview: 5+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2023-10-27 08:34 pgsql: Adjust parallel_schedule with event triggers on authenticated lo Alexander Korotkov <akorotkov@postgresql.org>
2023-10-27 10:47 ` Alvaro Herrera <alvherre@alvh.no-ip.org>
2023-10-27 11:20 ` Alexander Korotkov <aekorotkov@gmail.com>
2023-10-27 11:51 ` Alvaro Herrera <alvherre@alvh.no-ip.org>
2023-10-27 20:53 ` Alexander Korotkov <aekorotkov@gmail.com>
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