MIME-Version: 1.0
References: <CAPmGK17dAZCXvwnfpr1eTfknTGdt=hYTV9405Gt5SqPOX8K84w@mail.gmail.com>
 <65af8fb4-4c45-9cc3-c6ec-80f3ac488e07@oss.nttdata.com> <CAPmGK17XczZmzU=0mhk2_oG9Omo1p-uZCNdZAh_3vuC6+YboiA@mail.gmail.com>
 <95066202-c7e6-e4be-05cc-bdceb45b68c7@oss.nttdata.com> <CAPmGK14rr1R_t4xxWyJVng04nDjGnxi=J5fz-TBNBH9Xn0OFYQ@mail.gmail.com>
 <2d7c480a-63c8-a244-0f2a-361e5097764f@oss.nttdata.com> <CAPmGK15FuPVGx3TGHKShsbPKKtF1y58-ZLcKoxfN-nqLj1dZ=g@mail.gmail.com>
 <d888307d-c5b2-88e7-e2c0-f2228ad0756b@oss.nttdata.com> <CAPmGK158hrd=ZfXmgkmNFHivgh18e4oE2Gz151C2Q4OBDjZ08A@mail.gmail.com>
 <3e57ebd7-b536-333e-c877-49a3ebb57079@oss.nttdata.com> <CAPmGK17nfEZkKhqoP8fAPKbnfSpf2JVgV5+_Xjv2mn8P-4X0hQ@mail.gmail.com>
 <2a2519b5-0c1f-87c8-5461-a4d8be0728e0@oss.nttdata.com> <66d99132-5946-ba74-ffa6-620eb50b7a8f@highgo.ca>
In-Reply-To: <66d99132-5946-ba74-ffa6-620eb50b7a8f@highgo.ca>
From: Etsuro Fujita <etsuro.fujita@gmail.com>
Date: Mon, 21 Feb 2022 14:55:07 +0900
Message-ID: <CAPmGK16m4ZPp4Gux2tF-nkXSbyv-Apie1xCT7AGBS1FzwHXLmA@mail.gmail.com>
Subject: Re: postgres_fdw: commit remote (sub)transactions in parallel during pre-commit
To: David Zhang <david.zhang@highgo.ca>
Cc: Fujii Masao <masao.fujii@oss.nttdata.com>, 
	PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Archived-At: <https://www.postgresql.org/message-id/CAPmGK16m4ZPp4Gux2tF-nkXSbyv-Apie1xCT7AGBS1FzwHXLmA%40mail.gmail.com>
Precedence: bulk

On Sat, Feb 19, 2022 at 6:55 AM David Zhang <david.zhang@highgo.ca> wrote:
> Tried to apply the patches to master branch, no warning found and
> regression test passed.

Thanks for testing!

> Now, we have many places (5) calling the same function with a constant
> number 30000. Is this a good time to consider redefine this number a
> macro somewhere?

Yeah, I think that is a good idea.  I=E2=80=99ll do so in the next version =
of
the parallel-abort patch (#0003) if no objections.

Best regards,
Etsuro Fujita