public inbox for [email protected]
help / color / mirror / Atom feedFrom: Nikolay Shaplov <[email protected]>
To: Daniel Gustafsson <[email protected]>
To: Andres Freund <[email protected]>
Cc: Dagfinn Ilmari Mannsåker <[email protected]>
Cc: PostgreSQL Developers <[email protected]>
Cc: Andrew Dunstan <[email protected]>
Cc: Tom Lane <[email protected]>
Cc: Peter Eisentraut <[email protected]>
Subject: Re: TAP output format in pg_regress
Date: Mon, 28 Nov 2022 22:02:38 +0300
Message-ID: <5348601.u5redSiXY4@thinkpad-pgpro> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
<[email protected]>
<[email protected]>
В письме от понедельник, 28 ноября 2022 г. 21:28:48 MSK пользователь Andres
Freund написал:
> On 2022-11-28 14:13:16 +0100, Daniel Gustafsson wrote:
> > > On 27 Nov 2022, at 11:22, Nikolay Shaplov <[email protected]> wrote:
> > > В письме от суббота, 26 ноября 2022 г. 23:35:45 MSK пользователь Daniel
> > > Gustafsson написал:
> > >
> > > I wold suggest to use word immediate instead of noatexit. This will do
> > > the
> > > code much more sensible for me.
> >
> > I think noatexit is clearer since the codepath is specifically to avoid
> > any
> > registered atexit functions. The point of this function is to be able to
> > call bail while in a function registered with atexit() so I think the
> > current name is better.
>
> +1
Ok, then I would not insist on it.
> > So this extra offset indicates that test is being included into parallel
> > group? Guess it not really obvious...
>
> Grouping parallel tests via an initial list of test and then indenting each
> test with whitespace was committed 22 years ago. While there might be
> better
> ways to do this, the lack of complaints so far at least seems to indicate
> that it isn't all too terrible.
Ok, it was there for 22 years, it will do no harm if it is left this way for
some time :-)
----
From my reviewer's point of view patch is ready for commit.
Thank you for your patience :-)
PS Should I change commitfest status?
--
Nikolay Shaplov aka Nataraj
Fuzzing Engineer at Postgres Professional
Matrix IM: @dhyan:nataraj.su
Attachments:
[application/pgp-signature] signature.asc (488B, ../5348601.u5redSiXY4@thinkpad-pgpro/2-signature.asc)
download
view thread (40+ messages) latest in thread
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: [email protected]
Cc: [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected]
Subject: Re: TAP output format in pg_regress
In-Reply-To: <5348601.u5redSiXY4@thinkpad-pgpro>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox