Re: Error "initial slot snapshot too large" in create replication slot - Mailing list pgsql-hackers

From Julien Rouhaud
Subject Re: Error "initial slot snapshot too large" in create replication slot
Date
Msg-id 20220112103855.ib4455yyebaxkuwp@jrouhaud
Whole thread Raw
In response to Re: Error "initial slot snapshot too large" in create replication slot  (Dilip Kumar <dilipbalaut@gmail.com>)
Responses Re: Error "initial slot snapshot too large" in create replication slot
List pgsql-hackers
Hi,

On Tue, Nov 02, 2021 at 04:40:39PM +0530, Dilip Kumar wrote:
> 
> I have fixed this, the updated patch is attached.

The cfbot reports that this patch doesn't compile:
https://cirrus-ci.com/task/5642000073490432?logs=build

[03:01:24.477] snapbuild.c: In function ‘SnapBuildInitialSnapshot’:
[03:01:24.477] snapbuild.c:587:2: error: ‘newsubxcnt’ undeclared (first use in this function); did you mean
‘newsubxip’?
[03:01:24.477]   587 |  newsubxcnt = 0;
[03:01:24.477]       |  ^~~~~~~~~~
[03:01:24.477]       |  newsubxip
[03:01:24.477] snapbuild.c:587:2: note: each undeclared identifier is reported only once for each function it appears
in
[03:01:24.477] snapbuild.c:535:8: warning: unused variable ‘maxxidcnt’ [-Wunused-variable]
[03:01:24.477]   535 |  int   maxxidcnt;
[03:01:24.477]       |        ^~~~~~~~~

Could you send a new version?  In the meantime I will switch the patch to
Waiting on Author.



pgsql-hackers by date:

Previous
From: Dave Page
Date:
Subject: Re: sepgsql logging
Next
From: Pavel Borisov
Date:
Subject: Stream replication test fails of cfbot/windows server 2019