On 4/20/17 14:29, Petr Jelinek wrote:
> +             /* Find unused worker slot. */
> +             if (!w->in_use)
>               {
> -                     worker = &LogicalRepCtx->workers[slot];
> -                     break;
> +                     worker = w;
> +                     slot = i;
> +             }

Doesn't this still need a break?  Otherwise it always picks the last slot.

-- 
Peter Eisentraut              http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to