Hi,

Thanks for the review!

On Sat, 12 Sept 2026 at 14:46, Pierre Forstmann
<[email protected]> wrote:
>
> I think that the that SQLSTATE chosen in unicode_assigned() is wrong: this 
> not a SQL statement syntax error but more a feature not supported for the 
> current database encoding;
> ERRCODE_FEATURE_NOT_SUPPORTED is more appropriate.

Hmm, you are right, added that instead.

> Maybe you could add a regression test for tid.c ?

I've added it for now, but personally I don't think it is required.

(PS. This patch is intended for master only)

Attached v2.

Regards,
Ayush

Attachment: v2-0001-Fix-internal-errors-reachable-from-SQL.patch
Description: Binary data

Reply via email to