Horton

Re: SQL Property Graph Queries (SQL/PGQ)

Junwang Zhao <zhjwpku@gmail.com>

View thread
From: Junwang Zhao <zhjwpku@gmail.com>
To: Ashutosh Bapat <ashutosh.bapat.oss@gmail.com>
Cc: Vik Fearing <vik@postgresfriends.org>, Ajay Pal <ajay.pal.k@gmail.com>, Imran Zaheer <imran.zhir@gmail.com>, Peter Eisentraut <peter@eisentraut.org>, pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2025-02-23T12:54:53Z
Lists: pgsql-hackers

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →

  1. Fix some typos and make small stylistic improvements

  2. Cleanup users and roles in graph_table_rls test

  3. Dump labels in reproducible order

  4. SQL Property Graph Queries (SQL/PGQ)

  5. Factor out constructSetOpTargetlist() from transformSetOperationTree()

  6. Sort out table_open vs. relation_open in rewriter

  7. Rename grammar nonterminal to simplify reuse

  8. Make ecpg parse.pl more robust with braces

  9. Don't lock partitions pruned by initial pruning

  10. Remove pg_regex_collation

  11. Use auxv to check for CRC32 instructions on ARM.

  12. Fix inappropriate uses of atol()

  13. Remove unnecessary array object_classes[] in dependency.c

Attachments

On Mon, Feb 10, 2025 at 11:00 PM Junwang Zhao <zhjwpku@gmail.com> wrote:
>
> Hi Ashutosh,
>
> On Mon, Feb 10, 2025 at 2:14 PM Ashutosh Bapat
> <ashutosh.bapat.oss@gmail.com> wrote:
> >
> > On Thu, Feb 6, 2025 at 8:22 PM Junwang Zhao <zhjwpku@gmail.com> wrote:
> > > > >
> > > >
> > > > I see you have added some negative tests - object not found tests, but
> > > > I didn't see positive tests. Hence I haven't added those changes in
> > > > the attached patchset. But we certainly need those changes. You may
> > > > want to submit a patch with positive tests. That code needs to be
> > > > fixed before committing anyway.
> > >
> > > The attached file adds the positive tests.
> >
> > Hi Junwang,
> > Thanks for the patch, but please post it as a separate patch in the
> > full patch-set, otherwise CFBot gets confused.
>
> Ok, see the attached.
>
> 0001-0009 are the original patches of 20250127 with rebase of master
> 0010 fix ci error due to `Show more-intuitive titles for psql
> commands`, see a14707da564e
> 0011 fix ci error due to unstable collation tests, we should not
> depend on pg_catalog."default", I observed 002_pg_upgrade.pl failure
> caused by this.
> 0012 trivial refactor of property graph object address
>
> >
> > --
> > Best Wishes,
> > Ashutosh Bapat
>
>
>
> --
> Regards
> Junwang Zhao

Here is v11 with another trivial refactor, I add a seperate patch file 0013, in
insert_property_record, there is no need to check `type`, `typmode` and
`collation` if the property doesn't exists before, in AlterPropGraph, there is
no need to call `check_all_labels_properties` for each added vertex or edge.

Other files remain unchanged except I’ve added some missing document and
typo fix we discussed in the list but not included in the previous
patch, I included
them in 0008.

-- 
Regards
Junwang Zhao