[PATCH] Clean up property graph error messages - Mailing list pgsql-hackers

From Ayush Tiwari
Subject [PATCH] Clean up property graph error messages
Date
Msg-id CAJTYsWXFy1j_T82+M_S9kFxU414tQYnZQD-b82=oL_LbG_5fPQ@mail.gmail.com
Whole thread
Responses Re: [PATCH] Clean up property graph error messages
Re: [PATCH] Clean up property graph error messages
List pgsql-hackers
Hi,

While looking at the SQL/PGQ property graph error paths, I noticed a
few small cleanups in propgraphcmds.c.

The attached patch fixes a user-visible error message from "mismatching
properties names" to "mismatching property names", and moves a
ReleaseSysCache() call before an ERROR ereport in
check_element_properties().  

The existing code should be cleaned up by
the resource owner on the ERROR path, but the explicit ReleaseSysCache()
placed after ereport(ERROR) was unreachable.

The regression expected output is updated for the changed error text.

[On a separate note, it might be better to change all "vertexes" to "vertices",
haven't included that in the patch though since the other one is not exactly
wrong?]

Regards,
Ayus
Attachment

pgsql-hackers by date:

Previous
From: Alexander Borisov
Date:
Subject: Re: Improve the performance of Unicode Normalization Forms.
Next
From: Daniel Gustafsson
Date:
Subject: Re: Serverside SNI support in libpq