Re: Error "initial slot snapshot too large" in create replication slot

From: Dilip Kumar <dilipbalaut(at)gmail(dot)com>
To: Julien Rouhaud <rjuju123(at)gmail(dot)com>
Cc: Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Error "initial slot snapshot too large" in create replication slot
Date: 2022-01-17 03:57:14
Message-ID: CAFiTN-vb_Mv+rRVDWtagvdkow-eWaT_xsGLSDiPfzL41UKiDwQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, Jan 12, 2022 at 4:09 PM Julien Rouhaud <rjuju123(at)gmail(dot)com> wrote:

> 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.
>

Thanks for notifying, I will work on this and send the update patch soon.

--
Regards,
Dilip Kumar
EnterpriseDB: http://www.enterprisedb.com

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Dilip Kumar 2022-01-17 03:59:33 Re: Add sub-transaction overflow status in pg_stat_activity
Previous Message Michael Paquier 2022-01-17 03:56:06 Re: Refactoring of compression options in pg_basebackup