Re: [HACKERS] tablesync patch broke the assumption that logical repdepends on? - Mailing list pgsql-hackers

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