Re: Cache invalidation after authentication (on-the-fly role creation)

From: Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com>
To: Andres Freund <andres(at)anarazel(dot)de>
Cc: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Cache invalidation after authentication (on-the-fly role creation)
Date: 2018-07-04 09:37:41
Message-ID: CAEepm=0jV_o4_ZBZ=F0OT3BLk7O7q7Er-7gFfnGq5c-ERpH6Uw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, Jul 4, 2018 at 4:35 PM, Michael Paquier <michael(at)paquier(dot)xyz> wrote:
> I wanted to comment on that this morning but forgot as my mind was
> driven away by another problem. What if you used the Julien-Rouhaud's
> method of a custom script with only ";" used as query and -c? This
> won't run any queries, and will stress authentication.

Ok, I tried "pgbench -c 8 -j 8 -T 60 --connect -f empty.sql postgres"
where empty.sql contains just ";", and the numbers were noisy but
around 1160 TPS with or without a patch.

On Wed, Jul 4, 2018 at 4:35 PM, Andres Freund <andres(at)anarazel(dot)de> wrote:
> On 2018-07-04 16:25:18 +1200, Thomas Munro wrote:
>> PerformAuthentication(MyProcPort);
>> + AcceptInvalidationMessages();
>> InitializeSessionUserId(username, useroid);
>
> FWIW, a comment explaining why it's done there seems appropriate.

Done.

--
Thomas Munro
http://www.enterprisedb.com

Attachment Content-Type Size
0001-Call-AcceptInvalidationMessages-after-authenticat-v2.patch application/octet-stream 1.6 KB

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Pavel Raiskup 2018-07-04 09:54:22 [PATCH] btree_gist: fix union implementation for variable length columns
Previous Message Peter Eisentraut 2018-07-04 08:46:30 Re: CREATE TABLE .. LIKE .. EXCLUDING documentation