Feedback-ID: i0c21471d:Fastmail
User-Agent: Cyrus-JMAP/3.11.0-alpha0-456-gcd147058c-fm-hotfix-20240509.001-g0aad06e4
MIME-Version: 1.0
Message-Id: <61182a55-87e6-4019-9274-5f2cc5c38f31@app.fastmail.com>
In-Reply-To: <3222278.1716139817@sss.pgh.pa.us>
References: <3222278.1716139817@sss.pgh.pa.us>
Date: Sun, 19 May 2024 14:49:22 -0300
From: "Euler Taveira" <euler@eulerto.com>
To: "Tom Lane" <tgl@sss.pgh.pa.us>, pgsql-hackers@lists.postgresql.org
Cc: "Euler Taveira" <euler.taveira@enterprisedb.com>,
 "Peter Eisentraut" <peter@eisentraut.org>
Subject: Re: State of pg_createsubscriber
Content-Type: multipart/alternative;
 boundary=cb9fa49e594c4726927462eea6b912cc
Archived-At: <https://www.postgresql.org/message-id/61182a55-87e6-4019-9274-5f2cc5c38f31%40app.fastmail.com>
Precedence: bulk

--cb9fa49e594c4726927462eea6b912cc
Content-Type: text/plain

On Sun, May 19, 2024, at 2:30 PM, Tom Lane wrote:
> I'm fairly disturbed about the readiness of pg_createsubscriber.
> The 040_pg_createsubscriber.pl TAP test is moderately unstable
> in the buildfarm [1], and there are various unaddressed complaints
> at the end of the patch thread (pretty much everything below [2]).
> I guess this is good enough to start beta with, but it's far from
> being good enough to ship, IMO.  If there were active work going
> on to fix these things, I'd feel better, but neither the C code
> nor the test script have been touched since 1 April.

My bad. :( I'll post patches soon to address all of the points.


--
Euler Taveira
EDB   https://www.enterprisedb.com/

--cb9fa49e594c4726927462eea6b912cc
Content-Type: text/html
Content-Transfer-Encoding: quoted-printable

<!DOCTYPE html><html><head><title></title><style type=3D"text/css">p.Mso=
Normal,p.MsoNoSpacing{margin:0}</style></head><body><div>On Sun, May 19,=
 2024, at 2:30 PM, Tom Lane wrote:<br></div><blockquote type=3D"cite" id=
=3D"qt" style=3D""><div>I'm fairly disturbed about the readiness of pg_c=
reatesubscriber.<br></div><div>The 040_pg_createsubscriber.pl TAP test i=
s moderately unstable<br></div><div>in the buildfarm [1], and there are =
various unaddressed complaints<br></div><div>at the end of the patch thr=
ead (pretty much everything below [2]).<br></div><div>I guess this is go=
od enough to start beta with, but it's far from<br></div><div>being good=
 enough to ship, IMO.&nbsp; If there were active work going<br></div><di=
v>on to fix these things, I'd feel better, but neither the C code<br></d=
iv><div>nor the test script have been touched since 1 April.<br></div></=
blockquote><div><br></div><div>My bad. :( I'll post patches soon to addr=
ess all of the points.<br></div><div><br></div><div><br></div><div id=3D=
"sig112409080"><div class=3D"signature">--<br></div><div class=3D"signat=
ure">Euler Taveira<br></div><div class=3D"signature">EDB&nbsp;&nbsp; <a =
href=3D"https://www.enterprisedb.com/">https://www.enterprisedb.com/</a>=
<br></div></div><div><br></div></body></html>
--cb9fa49e594c4726927462eea6b912cc--