Re: pgsql: Some refactoring of logical/worker.c

On 2020-03-24 15:17, Peter Eisentraut wrote:
> Some refactoring of logical/worker.c
> 
> Discussion:
> https://www.postgresql.org/message-id/flat/CA+HiwqH=Y85vRK3mOdjEkqFK+E=ST=eQiHdpj43L=_eJMOOznQ@mail.gmail.com
> 
> Branch
> ------
> master
> 
> Details
> -------
> https://git.postgresql.org/pg/commitdiff/97ee604d9bcedf0066986521a7ee15e3aa87be34
> 
> Modified Files
> --------------
> src/backend/replication/logical/worker.c | 174 
> ++++++++++++++++++++-----------
> 1 file changed, 115 insertions(+), 59 deletions(-)

gcc 9.3.0 mutters under its breath,

worker.c: In function ‘apply_handle_update_internal’:
worker.c:791:22: warning: unused variable ‘remoterel’ 
[-Wunused-variable]
   791 |  LogicalRepRelation *remoterel = &relmapentry->remoterel;
       |

Otherwise the build is fine (documentation build too, including .pdf, 
thanks Tom)


Erik Rijkers