Use locale-aware value for \watch in 005_timeouts.pl
Reported-by: Alexander Lakhin
Branch
--
master
Details
---
https://git.postgresql.org/pg/commitdiff/605062227fdc8331f2ffa0f60e7a8724b6e56480
Modified Files
--
src/test/modules/test_misc/t/005_timeouts.pl | 18 +++--
Fix handling of expecteddir in pg_regress
Commit c855872074b introduced a new parameter to pg_regress to set
the directory where to look for expected files, but accidentally
only implemented it for when compiling pg_regress for ECPG tests.
Fix by adding support for the parameter to the main regres
Fix handling of expecteddir in pg_regress
Commit c855872074b introduced a new parameter to pg_regress to set
the directory where to look for expected files, but accidentally
only implemented it for when compiling pg_regress for ECPG tests.
Fix by adding support for the parameter to the main regres
Fix backstop in gin test if injection point is not reached
Per Tom Lane's observation that the test got stuck in infinite loop if
the injection_points module was not loaded. It was supposed to give up
after 1 iterations, but the backstop was broken.
Discussion:
https://www.postgresql.org/mes
Try to unbreak injection-fault tests in the buildfarm
The buildfarm script attempts to run all tests marked as
NO_INSTALLCHECK under src/test/modules without paying attention to
whether they are enabled or disabled in the parent Makefile. That
hasn't been a problem so far, because all the tests ma
On Fri, Mar 15, 2024 at 2:25 PM Andrey M. Borodin wrote:
> > On 15 Mar 2024, at 16:30, Alexander Korotkov wrote:
> >
> > Maybe, but do you see any negative side effects of the unconditionally
> > unset of flags?
>
> Nope, just expressed possible option.
>
> > If not, I would prefer to keep the c
Fix wordings in timeouts TAP test
Reported-by: Kyotaro Horiguchi
Discussion:
https://postgr.es/m/20240315.104235.1835366724413653745.horikyota.ntt%40gmail.com
Author: Andrey Borodin
Branch
--
master
Details
---
https://git.postgresql.org/pg/commitdiff/7a65cc079e717c5d87f1920929affe7138c
Fix race condition in transaction timeout TAP tests
The interruption handler within the injection point can get stuck in an
infinite loop while handling transaction timeout. To avoid this situation
we reset the timeout flag before invoking the injection point.
Author: Alexander Korotkov
Reviewed-
> On 15 Mar 2024, at 16:30, Alexander Korotkov wrote:
>
> Maybe, but do you see any negative side effects of the unconditionally
> unset of flags?
Nope, just expressed possible option.
> If not, I would prefer to keep the code simple.
IMO that's fine. Let's incorporate wording improvement f
On Fri, Mar 15, 2024 at 1:27 PM Andrey M. Borodin wrote:
>
> > On 15 Mar 2024, at 15:44, Alexander Korotkov wrote:
> >
> > We loop in the interrupt checking, given that the injection point handler
> > checks for interrupts internally. I propose to unset the timeout flag
> > before the injecti
> On 15 Mar 2024, at 15:44, Alexander Korotkov wrote:
>
> We loop in the interrupt checking, given that the injection point handler
> checks for interrupts internally. I propose to unset the timeout flag before
> the injection point (see the attached patch).
Oh, cool.
As far as I understa
Improve log messages referring to background worker processes
"Worker" could also mean autovacuum worker or slot sync worker, so
let's be more explicit.
Per Tristan Partin's suggestion.
Discussion:
https://www.postgresql.org/message-id/czm6wdx5h4qi.nzg1yuck...@neon.tech
Branch
--
master
D
Disable tests using injection points in installcheck
The 'gin' test injections faults to GIN index build. If another test
running concurrently in the same cluster also tries to create a GIN
index, it will hit the fault, too.
To fix, disable tests using injection points when running against an
exi
On Fri, Mar 15, 2024 at 9:44 AM Michael Paquier wrote:
> On Fri, Mar 15, 2024 at 11:20:49AM +0500, Andrey M. Borodin wrote:
> > And hachi sometimes pass this test too [0].
> >
> > I’ll look more on this. Do I understand right that we have only 2
buildfarm members with injection points?
>
> I have
On Fri, Mar 15, 2024 at 11:20:49AM +0500, Andrey M. Borodin wrote:
> And hachi sometimes pass this test too [0].
>
> I’ll look more on this. Do I understand right that we have only 2 buildfarm
> members with injection points?
I have added --enable-injection-points to three of them: hachi,
gokibu
15 matches
Mail list logo