Re: Improve the error message for logical replication of regular column to generated column. - Mailing list pgsql-hackers

From Amit Kapila
Subject Re: Improve the error message for logical replication of regular column to generated column.
Date
Msg-id CAA4eK1L5g1FjhnwTHpQ50oYND_=3dwXx4J7Bf4bo8oxbYJbgjA@mail.gmail.com
Whole thread Raw
In response to Re: Improve the error message for logical replication of regular column to generated column.  (vignesh C <vignesh21@gmail.com>)
List pgsql-hackers
On Wed, Nov 27, 2024 at 12:45 PM vignesh C <vignesh21@gmail.com> wrote:
>
> >
> > There is a buildfarm failure in [1]. One of the new tests added to
> > verify the log for the "incompatible generated columns" issue was
> > incorrect. Specifically, the check qr/ERROR: ( [A-Z0-9]:) should have
> > been updated to qr/ERROR: ( [A-Z0-9]+:), which is consistent with
> > similar checks elsewhere in the codebase. The attached patch contains
> > the necessary changes to address this issue.
> > https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=prion&dt=2024-11-27%2004%3A17%3A03
>
> The issue occurs specifically on the prion machine, which is
> configured with log_error_verbosity = verbose, causing error messages
> to include the sqlerrcode alongside the error description, as shown
> below from [1]:
> 2024-11-27 05:41:13.966 UTC [2990900:3] ERROR:  55000: logical
> replication target relation "public.t1" has incompatible generated
> columns: "c2", "c3"
>
> In contrast, other buildfarm machines do not include the sqlerrcode in
> the error messages, as seen here from [2]:
> 2024-11-27 07:19:45.975 CET [38683:2] ERROR:  logical replication
> target relation "public.t1" has incompatible generated columns: "c2",
> "c3"
>
> The problem arises only when the sqlerrcode is present, as the error
> code matching was not correct. I have confirmed that the patch
> referenced in [3] resolves the issue when log_error_verbosity =
> verbose is enabled.
>

Thanks for the analysis. I have pushed your fix.

--
With Regards,
Amit Kapila.



pgsql-hackers by date:

Previous
From: wenhui qiu
Date:
Subject: Re: An inefficient query caused by unnecessary PlaceHolderVar
Next
From: Kirill Reshke
Date:
Subject: Re: Add Pipelining support in psql