public inbox for [email protected]
help / color / mirror / Atom feedFrom: Amit Kapila <[email protected]>
To: [email protected] <[email protected]>
Cc: [email protected] <[email protected]>
Cc: Dilip Kumar <[email protected]>
Cc: Masahiko Sawada <[email protected]>
Cc: [email protected] <[email protected]>
Cc: Peter Smith <[email protected]>
Cc: PostgreSQL Hackers <[email protected]>
Cc: [email protected] <[email protected]>
Subject: Re: Perform streaming logical transactions by background workers and parallel apply
Date: Thu, 13 Oct 2022 09:19:26 +0530
Message-ID: <CAA4eK1+OBVyu5Vy3dCH_ofF7qBprdB9iq-71Pa1kynii_Ly0UQ@mail.gmail.com> (raw)
In-Reply-To: <TYAPR01MB5866C7ED7046BBFC1B8FDD26F55C9@TYAPR01MB5866.jpnprd01.prod.outlook.com>
References: <OS0PR01MB5716730C06159452335D870D947F9@OS0PR01MB5716.jpnprd01.prod.outlook.com>
<CAA4eK1Lu-6oXMk7ZaGYLwm3CRLBuzueGbasyHnNpJxu6Mq3mmg@mail.gmail.com>
<OS3PR01MB6275F145878B4A44586C46CE9E499@OS3PR01MB6275.jpnprd01.prod.outlook.com>
<CAA4eK1LMVdS6uM7Tw7ANL0BetAd76TKkmAXNNQa0haTe2tax6g@mail.gmail.com>
<OS3PR01MB6275EFC4B707650DAB9392859E4D9@OS3PR01MB6275.jpnprd01.prod.outlook.com>
<CAHut+PuVjRgGr4saN7qwq0oB8DANHVR7UfDiciB1Q3cYN54F6A@mail.gmail.com>
<OS3PR01MB627567CF4B96A23DAFAF77269E4E9@OS3PR01MB6275.jpnprd01.prod.outlook.com>
<TYAPR01MB5866727DCFC8B47FAC46ABF6F54E9@TYAPR01MB5866.jpnprd01.prod.outlook.com>
<OS3PR01MB6275F4A7CA186412E2FF2ED29E529@OS3PR01MB6275.jpnprd01.prod.outlook.com>
<TYAPR01MB586669F3243F2F399712FC98F5529@TYAPR01MB5866.jpnprd01.prod.outlook.com>
<TYAPR01MB58662D8CEC97E774FCE62F05F5559@TYAPR01MB5866.jpnprd01.prod.outlook.com>
<OS0PR01MB57162BC6B0FB848F216410DE94559@OS0PR01MB5716.jpnprd01.prod.outlook.com>
<TYAPR01MB58664D1D5B40689769386873F5579@TYAPR01MB5866.jpnprd01.prod.outlook.com>
<TYAPR01MB5866C7ED7046BBFC1B8FDD26F55C9@TYAPR01MB5866.jpnprd01.prod.outlook.com>
On Thu, Oct 6, 2022 at 6:09 PM [email protected]
<[email protected]> wrote:
>
> ~~~
> 10. worker.c - apply_handle_stream_start
>
> ```
> + *
> + * XXX We can avoid sending pair of the START/STOP messages to the parallel
> + * worker because unlike apply worker it will process only one
> + * transaction-at-a-time. However, it is not clear whether that is worth the
> + * effort because it is sent after logical_decoding_work_mem changes.
> ```
>
> I can understand that START message is not needed, but is STOP really removable? If leader does not send STOP to its child, does it lose a chance to change the worker-state to IDLE_IN_TRANSACTION?
>
I think if we want we can set that state before we went to sleep in
parallel apply worker. So, I guess ideally we don't need both of these
messages but for now, it is fine as mentioned in the comments.
--
With Regards,
Amit Kapila.
view thread (625+ messages) latest in thread
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected]
Subject: Re: Perform streaming logical transactions by background workers and parallel apply
In-Reply-To: <CAA4eK1+OBVyu5Vy3dCH_ofF7qBprdB9iq-71Pa1kynii_Ly0UQ@mail.gmail.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox