| From: | Ashutosh Bapat <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com> |
|---|---|
| To: | Peter Eisentraut <peter(at)eisentraut(dot)org> |
| Cc: | assam258(at)gmail(dot)com, Amit Langote <amitlangote09(at)gmail(dot)com>, Junwang Zhao <zhjwpku(at)gmail(dot)com>, Vik Fearing <vik(at)postgresfriends(dot)org>, Ajay Pal <ajay(dot)pal(dot)k(at)gmail(dot)com>, Imran Zaheer <imran(dot)zhir(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: SQL Property Graph Queries (SQL/PGQ) |
| Date: | 2026-02-27 17:06:13 |
| Message-ID: | CAExHW5uQAiB7OrYbr=tzpdjA3R-zrqPAr_rZ2wa+nvV32xk9fg@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Fri, Feb 27, 2026 at 2:14 PM Ashutosh Bapat
<ashutosh(dot)bapat(dot)oss(at)gmail(dot)com> wrote:
> >
> > Except, this error message change
> >
> > -ERROR: property "ename" of element variable "src" not found
> > +ERROR: none of the property graph elements associated with variable
> > "src" have property with name "ename" defined
> >
> > I don't know, that seems quite a complicated message for such a simple
> > mistake. Also, properties are associated with labels, not with graph
> > elements, so this seems misleading.
> >
>
> Hmm, mentioning labels was getting even more complicated, so I had to drop it.
>
> How about the earlier version with "of" replaced with "for"? "property
> "ename" for element variable "src" not found. The users can lookup the
> labels associated with variable src and figure out what went wrong
> themselves? I want to avoid simplistic error that other products give
> like "property "ename" not found" since it does not clarify whether
> the property is absent in the property graph or is not associated with
> labels in the pattern.
Here's a patchset with all the earlier patches squashed into 0001.
0002 is changing this specific error message as described in the para
above. Please let me know if you have any other suggestions.
--
Best Wishes,
Ashutosh Bapat
| Attachment | Content-Type | Size |
|---|---|---|
| v20260227-0002-Simplify-property-not-found-error-message.patch | text/x-patch | 4.1 KB |
| v20260227-0001-WIP-SQL-Property-Graph-Queries-SQL-PGQ.patch | text/x-patch | 709.0 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Andres Freund | 2026-02-27 17:32:24 | Re: Release postmaster working memory context in slotsync worker |
| Previous Message | Fujii Masao | 2026-02-27 17:00:17 | Re: doc: Clarify that empty COMMENT string removes the comment |