Re: SQL Property Graph Queries (SQL/PGQ)

Ashutosh Bapat <ashutosh.bapat.oss@gmail.com>

From: Ashutosh Bapat <ashutosh.bapat.oss@gmail.com>

To: Andreas Karlsson <andreas@proxel.se>

Cc: 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-07-29T05:03:46Z

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 →

Fix some typos and make small stylistic improvements
- 5282bf535e47 19 (unreleased) landed
Cleanup users and roles in graph_table_rls test
- 040a56be4bcc 19 (unreleased) landed
Dump labels in reproducible order
- c9babbc8816a 19 (unreleased) landed
SQL Property Graph Queries (SQL/PGQ)
- 2f094e7ac691 19 (unreleased) landed
Factor out constructSetOpTargetlist() from transformSetOperationTree()
- 8c2b30487cc7 19 (unreleased) landed
Sort out table_open vs. relation_open in rewriter
- d537f59fbbfc 19 (unreleased) landed
Rename grammar nonterminal to simplify reuse
- 8080f44f96a9 19 (unreleased) landed
Make ecpg parse.pl more robust with braces
- 7f88553ceaca 19 (unreleased) landed
Don't lock partitions pruned by initial pruning
- 525392d5727f 18.0 cited
Remove pg_regex_collation
- 792b2c7e6d92 18.0 cited
Use auxv to check for CRC32 instructions on ARM.
- aac831cafa6f 18.0 cited
Fix inappropriate uses of atol()
- f5a1311fccd2 18.0 cited
Remove unnecessary array object_classes[] in dependency.c
- ef5e2e90859a 17.0 cited

Hi Andreas,

On Sun, Dec 22, 2024 at 3:15 AM Andreas Karlsson <andreas@proxel.se> wrote:
>
> On 10/29/24 8:55 PM, Andreas Karlsson wrote:
> > I especially dislike the static variable in our patch. And as far as I
> > understand it you can avoid the static by changing the lexer to use the
> > push parser so it can emit multiple terminal tokens from one parsed
> > token, but I have not looked into push parsers and have no idea how this
> > would affect performance.
>
> Updated the patch to remove the static variable. No clue why I thought
> that one was necessary.

I have not included this patch in the latest patchset. Given that
Peter E has written the grammar, it best be handled by him. Please
feel free to post it again as an additional patch in the latest
patchset. A standalone patch like this confused CI Bot.

-- 
Best Wishes,
Ashutosh Bapat