supabase / pg_graphql

GraphQL support for PostgreSQL
https://supabase.github.io/pg_graphql
Apache License 2.0
2.88k stars 101 forks source link

Bumps pgrx to support PG17 #552

Closed davidgomes closed 6 days ago

davidgomes commented 1 week ago

What kind of change does this PR introduce?

This PR bumps the pgrx version to 0.12.4 so that PG17 can be supported.

What is the new behavior?

$ cargo pgrx run pg17
    Starting Postgres v17 on port 28817
     Creating database pg_graphql
psql (17rc1)
Type "help" for help.

david@pg_graphql=# create extension pg_graphql;
CREATE EXTENSION
Time: 371.664 ms
david@pg_graphql=# create table book(id int primary key, title text);
CREATE TABLE
Time: 16.246 ms
david@pg_graphql=# insert into book(id, title) values (1, 'book 1');
INSERT 0 1
Time: 3.510 ms
david@pg_graphql=# select graphql.resolve($$
[more] $> query {
[more] $>   bookCollection {
[more] $>     edges {
[more] $>       node {
[more] $>         id
[more] $>       }
[more] $>     }
[more] $>   }
[more] $> }
[more] $> $$);
┌────────────────────────────────────────────────────────────────┐
│                            resolve                             │
├────────────────────────────────────────────────────────────────┤
│ {"data": {"bookCollection": {"edges": [{"node": {"id": 1}}]}}} │
└────────────────────────────────────────────────────────────────┘
(1 row)

Time: 109.411 ms
david@pg_graphql=# ^D\q

(I tested the same with cargo pgrx run pg16)

imor commented 1 week ago

Thanks for the PR @davidgomes, could you please also bump pgrx at these locations:

olirice commented 6 days ago

thanks!