Re: [PATCH] Clean up property graph error messages

From: Peter Eisentraut <peter(at)eisentraut(dot)org>
To: Ayush Tiwari <ayushtiwari(dot)slg01(at)gmail(dot)com>, Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com>
Cc: PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>, Ashutosh Bapat <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com>
Subject: Re: [PATCH] Clean up property graph error messages
Date: 2026-05-07 08:46:04
Message-ID: 819e754b-2f8d-4c3d-8f9c-c25965dfea5f@eisentraut.org
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 06.05.26 08:23, Ayush Tiwari wrote:
> On Tue, 5 May 2026 at 08:20, Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com
> <mailto:li(dot)evan(dot)chao(at)gmail(dot)com>> wrote:
>
>
> So, v1 LGTM. Only thing is that I think we can also make
> “mismatching” to “mismatched” in the errmsg.
>
> I agree that "mismatched property names" reads better, so I have updated
> that in v2.

I think the existing wording is better. In any case, you only changed a
few places and not others.

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message vignesh C 2026-05-07 08:59:46 Re: Proposal: Conflict log history table for Logical Replication
Previous Message Peter Eisentraut 2026-05-07 08:45:13 Re: [PATCH] Clean up property graph error messages