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

  1. Switch to multi-inserts when registering dependencies for many code paths

  2. Use multi-inserts for pg_depend