Re: Re: [PATCH] BUG #13416: Postgres =?utf-8?Q?>=3D_?=9.3 doesn't use optimized shared memory on Solaris anymore

From: Sean Chittenden <seanc(at)joyent(dot)com>
To: Andres Freund <andres(at)anarazel(dot)de>
Cc: pgsql-hackers(at)postgresql(dot)org, stadtkind2(at)gmx(dot)de, Sam Gwydir <sam(dot)gwydir(at)joyent(dot)com>, Sean Chittenden <sean(at)chittenden(dot)org>, pgsql-bugs(at)postgresql(dot)org
Subject: Re: Re: [PATCH] BUG #13416: Postgres =?utf-8?Q?>=3D_?=9.3 doesn't use optimized shared memory on Solaris anymore
Date: 2017-10-05 21:34:37
Message-ID: b0522138-40bb-4775-ad69-eb67fb7e3521@Spark
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs pgsql-hackers

Fair enough.  We observed a ~4x amplification in memory usage so it was rather severe in our case.

The patch you referenced was a much nicer approach and Sam updated it to match that style (thank you Sam!).  We debated this internally and feel strongly that this should be exposed as a runtime GUC as suggested.  What's your take on the attached patch?

-sc

--
Sean Chittenden
seanc(at)joyent(dot)com

On Oct 4, 2017, 10:56 AM -0700, Andres Freund <andres(at)anarazel(dot)de>, wrote:
> Hi,
>
> On 2017-10-04 10:47:06 -0700, Sean Chittenden wrote:
> > Hello.  We identified the same problem.  Sam Gwydir and Josh Clulow were able to put together the appropriate fix after.
> >
> > The breakage in src/backend/port/sysv_shmem.c and src/include/storage/dsm_impl.h should be back ported to all supported versions (the regression was introduced between the 9.2 and 9.3 branches).
>
> Personally I don't think "breakage" is quite the right work.
>
> I also don't like that we're unconditionally not using
> USE_ANONYMOUS_SHMEM - doesn't that run into similar config limits on
> solaris based stuff as it does on linux etc?
>
> I think if we want to do this, we should rather go with a patch like
> https://www.postgresql.org/message-id/20140422121921.GD4449@awork2.anarazel.de
>
> Greetings,
>
> Andres Freund

Attachment Content-Type Size
ism2.patch application/octet-stream 6.2 KB

In response to

Browse pgsql-bugs by date

  From Date Subject
Next Message Tom Lane 2017-10-05 21:43:39 Re: pg_logical_slot_peek_changes crashes postgres when called from inside pl/pgsql
Previous Message Andres Freund 2017-10-05 21:10:58 Re: pg_logical_slot_peek_changes crashes postgres when called from inside pl/pgsql

Browse pgsql-hackers by date

  From Date Subject
Next Message Robert Haas 2017-10-05 21:34:39 Re: Proposal for CSN based snapshots
Previous Message Robert Haas 2017-10-05 21:02:38 Re: parallelize queries containing initplans