Re: Questions about logicalrep_worker_launch()

2025-04-29 Thread Amit Kapila
On Mon, Apr 28, 2025 at 2:37 PM Fujii Masao wrote: > > On 2025/04/26 3:03, Masahiko Sawada wrote: > > I agree with these changes. > > > > I think that while the changes for (2) should be for v19, the changes > > for (1) might be treated as a bug fix? > > Agreed. I've split the patch into two parts

Re: Questions about logicalrep_worker_launch()

2025-04-28 Thread Fujii Masao
On 2025/04/26 3:03, Masahiko Sawada wrote: I agree with these changes. I think that while the changes for (2) should be for v19, the changes for (1) might be treated as a bug fix? Agreed. I've split the patch into two parts: 0001 is for (1) and is a bug fix that should be back-patched to v1

Re: Questions about logicalrep_worker_launch()

2025-04-25 Thread Masahiko Sawada
Hi, On Fri, Apr 25, 2025 at 9:10 AM Fujii Masao wrote: > > Hi, > > While reading the code of logicalrep_worker_launch(), I had two questions: > > (1) > When the sync worker limit per subscription is reached, > logicalrep_worker_launch() > runs garbage collection to try to free up slots before ch

Questions about logicalrep_worker_launch()

2025-04-25 Thread Fujii Masao
Hi, While reading the code of logicalrep_worker_launch(), I had two questions: (1) When the sync worker limit per subscription is reached, logicalrep_worker_launch() runs garbage collection to try to free up slots before checking the limit again. That makes sense. But should we do the same whe