On Thu, Jan 12, 2023 at 8:25 AM Ted Yu wrote:
> On Wed, Jan 11, 2023 at 6:54 PM Amit Kapila wrote:
>>
>> On Wed, Jan 11, 2023 at 9:31 AM Ted Yu wrote:
>> >
>> > On Tue, Jan 10, 2023 at 7:55 PM houzj.f...@fujitsu.com
>> > wrote:
>> >>
>> >> On Wednesday, January 11, 2023 10:21 AM Ted Yu
>> >
On Wed, Jan 11, 2023 at 6:54 PM Amit Kapila wrote:
> On Wed, Jan 11, 2023 at 9:31 AM Ted Yu wrote:
> >
> > On Tue, Jan 10, 2023 at 7:55 PM houzj.f...@fujitsu.com <
> houzj.f...@fujitsu.com> wrote:
> >>
> >> On Wednesday, January 11, 2023 10:21 AM Ted Yu
> wrote:
> >> > /* First time thr
On Wed, Jan 11, 2023 at 9:31 AM Ted Yu wrote:
>
> On Tue, Jan 10, 2023 at 7:55 PM houzj.f...@fujitsu.com
> wrote:
>>
>> On Wednesday, January 11, 2023 10:21 AM Ted Yu wrote:
>> > /* First time through, initialize parallel apply worker state
>> > hashtable. */
>> > if (!Parallel
On Tue, Jan 10, 2023 at 7:55 PM houzj.f...@fujitsu.com <
houzj.f...@fujitsu.com> wrote:
> On Wednesday, January 11, 2023 10:21 AM Ted Yu
> wrote:
> > /* First time through, initialize parallel apply worker state
> hashtable. */
> > if (!ParallelApplyTxnHash)
> >
> > I think it wou
On Wednesday, January 11, 2023 10:21 AM Ted Yu wrote:
> /* First time through, initialize parallel apply worker state
> hashtable. */
> if (!ParallelApplyTxnHash)
>
> I think it would be better if `ParallelApplyTxnHash` is created by the first
> successful parallel apply worker.
On Tue, Jan 10, 2023 at 6:13 PM houzj.f...@fujitsu.com <
houzj.f...@fujitsu.com> wrote:
> On Wednesday, January 11, 2023 1:25 AM Ted Yu wrote:
>
> > I was reading src/backend/replication/logical/applyparallelworker.c .
> > In `pa_allocate_worker`, when pa_launch_parallel_worker returns NULL, I
>
On Wednesday, January 11, 2023 1:25 AM Ted Yu wrote:
> I was reading src/backend/replication/logical/applyparallelworker.c .
> In `pa_allocate_worker`, when pa_launch_parallel_worker returns NULL, I think
> the `ParallelApplyTxnHash` should be released.
Thanks for reporting.
ParallelApplyTxnHa
On Tue, Jan 10, 2023 at 9:43 AM Ted Yu wrote:
>
>
> On Tue, Jan 10, 2023 at 9:26 AM Ted Yu wrote:
>
>>
>>
>> On Tue, Jan 10, 2023 at 9:25 AM Ted Yu wrote:
>>
>>> Hi,
>>> I was reading src/backend/replication/logical/applyparallelworker.c .
>>> In `pa_allocate_worker`, when pa_launch_parallel_wo
On Tue, Jan 10, 2023 at 9:26 AM Ted Yu wrote:
>
>
> On Tue, Jan 10, 2023 at 9:25 AM Ted Yu wrote:
>
>> Hi,
>> I was reading src/backend/replication/logical/applyparallelworker.c .
>> In `pa_allocate_worker`, when pa_launch_parallel_worker returns NULL, I
>> think the `ParallelApplyTxnHash` shoul
On Tue, Jan 10, 2023 at 9:25 AM Ted Yu wrote:
> Hi,
> I was reading src/backend/replication/logical/applyparallelworker.c .
> In `pa_allocate_worker`, when pa_launch_parallel_worker returns NULL, I
> think the `ParallelApplyTxnHash` should be released.
>
> Please see the patch.
>
> Thanks
>
Here
10 matches
Mail list logo