Horton

Re: proposal: schema variables

Pavel Stehule <pavel.stehule@gmail.com>

View thread
From: Pavel Stehule <pavel.stehule@gmail.com>
To: Jim Jones <jim.jones@uni-muenster.de>
Cc: Bruce Momjian <bruce@momjian.us>, Dmitry Dolgov <9erthalion6@gmail.com>, Laurenz Albe <laurenz.albe@cybertec.at>, Erik Rijkers <er@xs4all.nl>, Michael Paquier <michael@paquier.xyz>, Amit Kapila <amit.kapila16@gmail.com>, DUVAL REMI <REMI.DUVAL@cheops.fr>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>, jian he <jian.universality@gmail.com>, Alvaro Herrera <alvherre@alvh.no-ip.org>, PegoraroF10 <marcos@f10.com.br>
Date: 2025-09-15T16:55:44Z
Lists: pgsql-hackers, pgsql-performance

Commits

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

  1. Move WAL sequence code into its own file

  2. Add ExplainState argument to pg_plan_query() and planner().

  3. Don't include access/htup_details.h in executor/tuptable.h

  4. Refactor to avoid code duplication in transformPLAssignStmt.

  5. Avoid including commands/dbcommands.h in so many places

  6. Restrict psql meta-commands in plain-text dumps.

  7. Split func.sgml into more manageable pieces

  8. Fix squashing algorithm for query texts

  9. EXPLAIN: Always use two fractional digits for row counts.

  10. Preliminary refactoring of plpgsql expression construction.

  11. plpgsql: pure parser and reentrant scanner

  12. Add some sanity checks in executor for query ID reporting

  13. Fix misleading error message context

  14. Add macros for looping through a List without a ListCell.

Attachments

Hi

po 15. 9. 2025 v 10:21 odesílatel Jim Jones <jim.jones@uni-muenster.de>
napsal:

> Hi
>
> On 9/13/25 11:28, Pavel Stehule wrote:
> >
> > minor change (after private talk with Jim Jones)
> >
>
> After another pass, there are a few additional tests that should be
> included in this patch:
>
>
> == Additional tests for GRANT x ON VARIABLE ==
> ==============================================
>
> We wanna make sure the proper error message is raised.
>
> postgres=# GRANT INSERT ON VARIABLE var TO jim;
> ERROR:  invalid privilege type INSERT for session variable
>
> postgres=# GRANT DELETE ON VARIABLE var TO jim;
> ERROR:  invalid privilege type DELETE for session variable
>
>
> == Tests for ALTER DEFAULT PRIVILEGES ==
> ========================================
>
> I couldn't find regression tests for ALTER DEFAULT PRIVILEGES ... if I
> haven't just missed them, I think they'd be a nice addition.
>
> postgres=# ALTER DEFAULT PRIVILEGES IN SCHEMA s GRANT UPDATE ON
> VARIABLES TO jim;
> ALTER DEFAULT PRIVILEGES
>
> postgres=# ALTER DEFAULT PRIVILEGES IN SCHEMA s GRANT INSERT ON
> VARIABLES TO jim;
> ERROR:  invalid privilege type INSERT for session variable
>
> postgres=# ALTER DEFAULT PRIVILEGES IN SCHEMA s GRANT DELETE ON
> VARIABLES TO jim;
> ERROR:  invalid privilege type DELETE for session variable
>
> postgres=# ALTER DEFAULT PRIVILEGES IN SCHEMA s GRANT SELECT ON
> VARIABLES TO jim;
> ALTER DEFAULT PRIVILEGES
>

done

Regards

Pavel


>
>
> Best, Jim
>