Re: Transactions involving multiple postgres foreign servers

From: Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>
To: Vinayak Pokale <pokale_vinayak_q3(at)lab(dot)ntt(dot)co(dot)jp>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Ashutosh Bapat <ashutosh(dot)bapat(at)enterprisedb(dot)com>
Subject: Re: Transactions involving multiple postgres foreign servers
Date: 2017-03-21 17:49:55
Message-ID: CAD21AoBo=ORuzjJpWSv9niNixrg2NokqDsjCRqSejDgT3DkD3Q@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox
Thread:
Lists: pgsql-hackers

On Thu, Mar 16, 2017 at 2:37 PM, Vinayak Pokale
<pokale_vinayak_q3(at)lab(dot)ntt(dot)co(dot)jp> wrote:
> The following review has been posted through the commitfest application:
> make installcheck-world: tested, passed
> Implements feature: tested, passed
> Spec compliant: tested, passed
> Documentation: tested, passed
>
> I have tested the latest patch and it looks good to me,
> so I marked it "Ready for committer".
> Anyway, it would be great if anyone could also have a look at the patches and send comments.
>
> The new status of this patch is: Ready for Committer
>

Thank you for updating but I found a bug in 001 patch. Attached latest patches.
The differences are
* Fixed a bug.
* Ran pgindent.
* Separated the patch supporting GetPrepareID API.

Regards,

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

Attachment Content-Type Size
004_regression_test_for_fdw_xact_v11.patch application/octet-stream 7.4 KB
005_get_prepare_id_api_v11.patch application/octet-stream 17.1 KB
003_fdw_transaction_resolver_contrib_v11.patch application/octet-stream 14.5 KB
002_pgfdw_support_atomic_commit_v11.patch application/octet-stream 42.0 KB
001_support_fdw_xact_v11.patch application/octet-stream 106.3 KB
000_register_local_write_v11.patch application/octet-stream 3.3 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Robert Haas 2017-03-21 17:54:39 Re: logical replication apply to run with sync commit off by default
Previous Message Robert Haas 2017-03-21 17:49:49 Re: [PATCH] Transaction traceability - txid_status(bigint)