From: Nikolay Shaplov <dhyan@nataraj.su>
To: Peter Smith <smithpb2250@gmail.com>
Cc: Alvaro Herrera <alvherre@alvh.no-ip.org>,
 Yura Sokolov <y.sokolov@postgrespro.ru>, pgsql-hackers@lists.postgresql.org,
 Michael Paquier <michael@paquier.xyz>
Subject: Re: [PATCH] New [relation] option engine
Date: Wed, 07 Feb 2024 09:44:52 +0300
Message-ID: <2312843.ElGaqSPkdT@thinkpad-pgpro>
Organization: Postgres Professional
In-Reply-To: <853387960.0ifERbkFSE@thinkpad-pgpro>
References: 
 <202312081259.2pwkb25emowf@alvherre.pgsql>
 <CAHut+PufgQPCxa9_mcco8+zapzB_18oOyFXuCY3jWmNUqt-CjQ@mail.gmail.com>
 <853387960.0ifERbkFSE@thinkpad-pgpro>
MIME-Version: 1.0
Content-Type: multipart/signed; boundary="nextPart1876656.tdWV9SEqCh";
 micalg="pgp-sha512"; protocol="application/pgp-signature"
Archived-At: 
 <https://www.postgresql.org/message-id/2312843.ElGaqSPkdT%40thinkpad-pgpro>
Precedence: bulk

--nextPart1876656.tdWV9SEqCh
Content-Type: multipart/mixed; boundary="nextPart4866644.GXAFRqVoOG";
 protected-headers="v1"
Content-Transfer-Encoding: 7Bit
From: Nikolay Shaplov <dhyan@nataraj.su>
To: Peter Smith <smithpb2250@gmail.com>
Subject: Re: [PATCH] New [relation] option engine
Date: Wed, 07 Feb 2024 09:44:52 +0300
Message-ID: <2312843.ElGaqSPkdT@thinkpad-pgpro>
Organization: Postgres Professional
In-Reply-To: <853387960.0ifERbkFSE@thinkpad-pgpro>
MIME-Version: 1.0

This is a multi-part message in MIME format.

--nextPart4866644.GXAFRqVoOG
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain; charset="UTF-8"

=D0=92 =D0=BF=D0=B8=D1=81=D1=8C=D0=BC=D0=B5 =D0=BE=D1=82 =D1=87=D0=B5=D1=82=
=D0=B2=D0=B5=D1=80=D0=B3, 1 =D1=84=D0=B5=D0=B2=D1=80=D0=B0=D0=BB=D1=8F 2024=
 =D0=B3. 09:58:32 MSK =D0=BF=D0=BE=D0=BB=D1=8C=D0=B7=D0=BE=D0=B2=D0=B0=D1=
=82=D0=B5=D0=BB=D1=8C Nikolay=20
Shaplov =D0=BD=D0=B0=D0=BF=D0=B8=D1=81=D0=B0=D0=BB:

I' ve updated the patch again, since the output of the=20
src/test/modules/test_oat_hooks test have changed.
This test began to register namespace lookups, and since options internals=
=20
have been changed much, number of lookups have been changed too. So I just=
=20
fixed the expected file.

> Hi!
>=20
> I've updated the patch, and it should work with modern master branch.
>=20
> > 2024-01 Commitfest.
> >=20
> > Hi, This patch has a CF status of "Needs Review" [1], but it seems
> > there were CFbot test failures last time it was run [2]. Please have a
> > look and post an updated version if necessary.
>=20
> Do we still miss notification facility for CFbot? May be somebody have ad=
ded
> it since the time I've checked last time?
>=20
> > =3D=3D=3D=3D=3D=3D
> > [1] https://commitfest.postgresql.org/46/4688/
> > [2] https://cirrus-ci.com/task/5066432363364352
> >=20
> > Kind Regards,
> > Peter Smith.


=2D-=20
Nikolay Shaplov aka Nataraj
=46uzzing Engineer at Postgres Professional
Matrix IM: @dhyan:nataraj.su

--nextPart4866644.GXAFRqVoOG
Content-Disposition: attachment; filename="v6-0001-New-options-engine.patch"
Content-Transfer-Encoding: 7Bit
Content-Type: text/x-patch; charset="unicode-2-0-utf-8";
 name="v6-0001-New-options-engine.patch"