On Fri, 7 Nov 2025 at 00:38, Mats Kindahl <mats.kindahl@gmail.com> wrote:
> As discussed in [1] the functions check_publications_origin_tables() and
> check_publications_origin_sequences() are building error messages using
> dynamically allocated StringInfo instances only to avoid duplicating a
> call of ereport().
Looks better and more traditional to me, plus git diff --stat reports:
1 file changed, 28 insertions(+), 33 deletions(-)
It has my vote. I've included Vignesh and Amit to see if they're also
ok with it.
David