On Tue, 18 Feb 2020 at 00:40, Muhammad Usama wrote:
>
> Hi Sawada San,
>
> I have a couple of comments on
> "v27-0002-Support-atomic-commit-among-multiple-foreign-ser.patch"
>
> 1- As part of the XLogReadRecord refactoring commit the signature of
> XLogReadRecord was changed,
> so the function
Hi Sawada San,
I have a couple of comments on
"v27-0002-Support-atomic-commit-among-multiple-foreign-ser.patch"
1- As part of the XLogReadRecord refactoring commit the signature of
XLogReadRecord was changed,
so the function call to XLogReadRecord() needs a small adjustment.
i.e. In function
On Wed, Sep 04, 2019 at 12:44:20PM +0900, Masahiko Sawada wrote:
> I forgot to include some new header files. Attached the updated patches.
No reviews since and the patch does not apply anymore. I am moving it
to next CF, waiting on author.
--
Michael
signature.asc
Description: PGP signature
Hello Sawada-san,
On 2019-Jul-02, Masahiko Sawada wrote:
> On Mon, Jul 1, 2019 at 8:32 PM Thomas Munro wrote:
> > Can we please have a fresh rebase?
>
> Thank you for the notice. Attached rebased patches.
... and again?
--
Álvaro Herrerahttps://www.2ndQuadrant.com/
PostgreSQ
On Wed, Apr 17, 2019 at 10:23 PM Masahiko Sawada wrote:
> Sorry for the very late. Attached updated version patches.
Hello Sawada-san,
Can we please have a fresh rebase?
Thanks,
--
Thomas Munro
https://enterprisedb.com
On Thu, Jan 31, 2019 at 11:09:09AM +0100, Masahiko Sawada wrote:
> Thanks. Actually I'm updating the patch set, changing API interface as
> I proposed before and improving the document and README. I'll submit
> the latest patch next week.
Cool, I have moved the patch to next CF.
--
Michael
signa
On Tue, Jan 29, 2019 at 5:47 PM Ildar Musin wrote:
>
> Hello,
>
> The patch needs rebase as it doesn't apply to the current master. I applied it
> to the older commit to test it. It worked fine so far.
Thank you for testing the patch!
>
> I found one bug though which would cause resolver to fini
Hello,
The patch needs rebase as it doesn't apply to the current master. I applied
it
to the older commit to test it. It worked fine so far.
I found one bug though which would cause resolver to finish by timeout even
though there are unresolved foreign transactions in the list. The
`fdw_xact_exis
On Thu, Nov 15, 2018 at 7:36 PM Masahiko Sawada wrote:
>
> On Mon, Oct 29, 2018 at 6:03 PM Masahiko Sawada wrote:
> >
> > On Mon, Oct 29, 2018 at 10:16 AM Masahiko Sawada
> > wrote:
> > >
> > > On Wed, Oct 24, 2018 at 9:06 AM Masahiko Sawada
> > > wrote:
> > > >
> > > > On Tue, Oct 23, 2018 a
On Tue, Oct 23, 2018 at 12:54 PM Kyotaro HORIGUCHI
wrote:
>
> Hello.
>
> # It took a long time to come here..
>
> At Fri, 19 Oct 2018 21:38:35 +0900, Masahiko Sawada
> wrote in
> > On Wed, Oct 10, 2018 at 1:34 PM Masahiko Sawada
> > wrote:
> ...
> > * Updated docs, added the new section "Dist
Hello.
# It took a long time to come here..
At Fri, 19 Oct 2018 21:38:35 +0900, Masahiko Sawada
wrote in
> On Wed, Oct 10, 2018 at 1:34 PM Masahiko Sawada wrote:
...
> * Updated docs, added the new section "Distributed Transaction" at
> Chapter 33 to explain the concept to users
>
> * Moved
On Wed, Oct 3, 2018 at 6:02 PM Chris Travers wrote:
>
>
>
> On Wed, Oct 3, 2018 at 9:41 AM Chris Travers wrote:
>>
>> The following review has been posted through the commitfest application:
>> make installcheck-world: tested, failed
>> Implements feature: not tested
>> Spec compliant:
On Wed, Oct 3, 2018 at 9:41 AM Chris Travers
wrote:
> The following review has been posted through the commitfest application:
> make installcheck-world: tested, failed
> Implements feature: not tested
> Spec compliant: not tested
> Documentation:tested, failed
>
> I
On Wed, Oct 3, 2018 at 9:56 AM Chris Travers
wrote:
>
>
> On Wed, Oct 3, 2018 at 9:41 AM Chris Travers
> wrote:
>
>>
>> (errmsg("preparing foreign transactions
> (max_prepared_foreign_transactions > 0) requires maX_foreign_xact_resolvers
> > 0")));
>
Two more critical notes here which I think a
On Wed, Oct 3, 2018 at 9:41 AM Chris Travers
wrote:
> The following review has been posted through the commitfest application:
> make installcheck-world: tested, failed
> Implements feature: not tested
> Spec compliant: not tested
> Documentation:tested, failed
>
Als
The following review has been posted through the commitfest application:
make installcheck-world: tested, failed
Implements feature: not tested
Spec compliant: not tested
Documentation:tested, failed
I am hoping I am not out of order in writing this before the commitfe
On Fri, Aug 03, 2018 at 05:52:24PM +0900, Masahiko Sawada wrote:
> I attached the updated version patch as the previous versions conflict
> with the current HEAD.
Please note that the latest patch set does not apply anymore, so this
patch is moved to next CF, waiting on author.
--
Michael
signat
17 matches
Mail list logo
