Re: [HACKERS] Transactions involving multiple postgres foreign servers

On Thu, Mar 2, 2017 at 11:56 AM, vinayak
<Pokale_Vinayak_q3@lab.ntt.co.jp> wrote:
>
> On 2017/02/28 16:54, Masahiko Sawada wrote:
>
> I've created a wiki page[1] describing about the design and
> functionality of this feature. Also it has some examples of use case,
> so this page would be helpful for even testing. Please refer it if
> you're interested in testing this feature.
>
> [1] 2PC on FDW
> <https://wiki.postgresql.org/wiki/2PC_on_FDW>
>
> Thank you for creating the wiki page.

Thank you for looking at this patch.

> In the "src/test/regress/pg_regress.c" file
> -                * xacts.  (Note: to reduce the probability of unexpected
> shmmax
> -                * failures, don't set max_prepared_transactions any higher
> than
> -                * actually needed by the prepared_xacts regression test.)
> +                * xacts. We also set max_fdw_transctions to enable testing
> of atomic
> +                * foreign transactions. (Note: to reduce the probability of
> unexpected
> +                * shmmax failures, don't set max_prepared_transactions or
> +                * max_prepared_foreign_transactions any higher than
> actually needed by the
> +                * corresponding regression tests.).
>
> I think we are not setting the "max_fdw_transctions" anywhere.
> Is this correct?

This comment is out of date. Will fix.

>
> In the "src/bin/pg_waldump/rmgrdesc.c" file following header file used two
> times.
> + #include "access/fdw_xact.h"
> I think we need to remove one line.
>

Not necessary. Will get rid of it.

Since these are not feature bugs I will incorporate these when making
update version patches.

Regards,

--
Masahiko Sawada
NIPPON TELEGRAPH AND TELEPHONE CORPORATION
NTT Open Source Software Center