Re: Proposal: "Causal reads" mode for load balancing reads without stale data - Mailing list pgsql-hackers

From Thomas Munro
Subject Re: Proposal: "Causal reads" mode for load balancing reads without stale data
Date
Msg-id CAEepm=18T1VdxufZRXRjcPMGe8FWtjVpU1cEtZ6JC7bgnW0WTg@mail.gmail.com
Whole thread Raw
In response to Re: Proposal: "Causal reads" mode for load balancing reads without stale data  (Robert Haas <robertmhaas@gmail.com>)
Responses Re: Proposal: "Causal reads" mode for load balancing reads without stale data
List pgsql-hackers
On Tue, Mar 15, 2016 at 6:58 AM, Robert Haas <robertmhaas@gmail.com> wrote:
> On Sun, Mar 13, 2016 at 11:50 PM, Thomas Munro
> <thomas.munro@enterprisedb.com> wrote:
>> The last patches I posted don't apply today due to changes in master,
>> so here's a freshly merged patch series.
>
> +        from the current synchronous stanbyindicates it has received the
>
> Uh, no.

Oops, thanks, fixed.  I'll wait for some more feedback or a conflict
with master before sending a new version.

> -    SyncRepWaitForLSN(gxact->prepare_end_lsn);
> +    {
> +        /*
> +         * Don't wait for the prepare to be applied remotely in remote_apply
> +         * mode, just wait for it to be flushed to the WAL.  We will wait for
> +         * apply when the transaction eventuallly commits or aborts.
> +         */
> +        if (synchronous_commit == SYNCHRONOUS_COMMIT_REMOTE_APPLY)
> +            assign_synchronous_commit(SYNCHRONOUS_COMMIT_REMOTE_FLUSH, NULL);
> +
> +        SyncRepWaitForLSN(gxact->prepare_end_lsn);
> +
> +        if (synchronous_commit == SYNCHRONOUS_COMMIT_REMOTE_APPLY)
> +            assign_synchronous_commit(SYNCHRONOUS_COMMIT_REMOTE_APPLY, NULL);
> +    }
>
> What's with the extra block?

Yeah, that's silly, thanks.  Tidied up for the next version.

-- 
Thomas Munro
http://www.enterprisedb.com



pgsql-hackers by date:

Previous
From: Robert Haas
Date:
Subject: Re: Obsolete comment in postgres_fdw.c
Next
From: Robert Haas
Date:
Subject: Re: [PROPOSAL] VACUUM Progress Checker.