Re: [HACKERS] A little improvementof ApplyLauncherMain loop code

2017-08-21 Thread Yugo Nagata
On Tue, 15 Aug 2017 15:17:06 -0400 Peter Eisentraut wrote: > On 8/1/17 02:28, Yugo Nagata wrote: > > When reading the logical replication code, I found that the following > > part could be improved a bit. In the foreach, LWLockAcquire and > >

Re: [HACKERS] A little improvementof ApplyLauncherMain loop code

2017-08-15 Thread Peter Eisentraut
On 8/1/17 02:28, Yugo Nagata wrote: > When reading the logical replication code, I found that the following > part could be improved a bit. In the foreach, LWLockAcquire and > logicalrep_worker_find are called for each loop, but they are needed > only when sub->enabled is true. Fixed, thanks! --

[HACKERS] A little improvementof ApplyLauncherMain loop code

2017-08-01 Thread Yugo Nagata
Hi, When reading the logical replication code, I found that the following part could be improved a bit. In the foreach, LWLockAcquire and logicalrep_worker_find are called for each loop, but they are needed only when sub->enabled is true. 846 /* Start the missing workers for enabled