Re: Switch to multi-inserts for pg_depend
Daniel Gustafsson <daniel@yesql.se>
From: Daniel Gustafsson <daniel@yesql.se>
To: Michael Paquier <michael@paquier.xyz>
Cc: Alvaro Herrera <alvherre@2ndquadrant.com>,
Robert Haas <robertmhaas@gmail.com>,
Andres Freund <andres@anarazel.de>,
Postgres hackers <pgsql-hackers@lists.postgresql.org>
Date: 2020-09-03T07:47:07Z
Lists: pgsql-hackers
> On 3 Sep 2020, at 07:35, Michael Paquier <michael@paquier.xyz> wrote: > > On Tue, Sep 01, 2020 at 11:53:36AM +0200, Daniel Gustafsson wrote: >> On 14 Aug 2020, at 20:23, Alvaro Herrera <alvherre@2ndquadrant.com> wrote: >> >>> The logic to keep track number of used slots used is baroque, though -- that >>> could use a lot of simplification. >> >> What if slot_init was an integer which increments together with the loop >> variable until max_slots is reached? If so, maybe it should be renamed >> slot_init_count and slotCount renamed slot_stored_count to make the their use >> clearer? > > Good idea, removing slot_init looks like a good thing for readability. > And the same can be done for pg_shdepend. I think this version is a clear improvement. Nothing more sticks out from a read-through. cheers ./daniel
Commits
-
Switch to multi-inserts when registering dependencies for many code paths
- 8febfd185545 14.0 landed
-
Use multi-inserts for pg_depend
- 63110c6264a5 14.0 landed