tcdi / plrust

A Rust procedural language handler for PostgreSQL
PostgreSQL License
1.12k stars 32 forks source link

Support for Postgres 17 #412

Open adamguo0 opened 3 months ago

adamguo0 commented 3 months ago

Hi, the Postgres community has cut the REL_17_STABLE branch. Do we know when PL/Rust (therefore pgrx) will support Postgres 17? Thanks.

$ cargo pgrx init --pg17 /usr/local/pgsql/bin/pg_config 
error: unexpected argument '--pg17' found

  tip: a similar argument exists: '--pg16'

Usage: cargo pgrx init <--pg11 <PG11>|--pg12 <PG12>|--pg13 <PG13>|--pg14 <PG14>|--pg15 <PG15>|--pg16 <PG16>|--base-port <BASE_PORT>|--base-testing-port <BASE_TESTING_PORT>|--configure-flag <CONFIGURE_FLAG>|--valgrind>

For more information, try '--help'.
eeeebbbbrrrr commented 3 months ago

pgrx has been going through some major work to help improve its overall soundness as it relates to managing Postgres-allocated memory. Once that work is complete, pl/rust will get a refresh.

Between now and then, I'm sure we'll get pg17 support added to pgrx proper, but that won't be reflected here in pl/rust until aforementioned tasks are complete.

It's on our radar -- if that's any comfort.

adamguo0 commented 3 months ago

Thanks, so the priority is to rework pgrx memory soundness before pg17 support will be available for plrust?

Do you have a rough idea of the timeline? say, before or after the planned PG17 release in September?

eeeebbbbrrrr commented 3 months ago

Thanks, so the priority is to rework pgrx memory soundness before pg17 support will be available for plrust?

Yes

Do you have a rough idea of the timeline? say, before or after the planned PG17 release in September?

No

Clearly our preference is for the stars to align, but work takes time.

adamguo0 commented 1 month ago

Hi @johnrballard, I understand that you're the maintainer of the PL/Rust project now. Is there any timeline on updating PL/Rust to support PG17?