From: | Andres Freund <andres(at)2ndquadrant(dot)com> |
---|---|
To: | Noah Misch <noah(at)leadboat(dot)com> |
Cc: | pgsql-committers(at)postgresql(dot)org |
Subject: | Re: pgsql: Permit super-MaxAllocSize allocations with MemoryContextAllocHug |
Date: | 2013-07-02 10:08:42 |
Message-ID: | 20130702100842.GA25626@alap2.anarazel.de |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Hi,
On 2013-06-27 19:05:22 +0000, Noah Misch wrote:
> Permit super-MaxAllocSize allocations with MemoryContextAllocHuge().
>
> The MaxAllocSize guard is convenient for most callers, because it
> reduces the need for careful attention to overflow, data type selection,
> and the SET_VARSIZE() limit. A handful of callers are happy to navigate
> those hazards in exchange for the ability to allocate a larger chunk.
> Introduce MemoryContextAllocHuge() and repalloc_huge(). Use this in
> tuplesort.c and tuplestore.c, enabling internal sorts of up to INT_MAX
> tuples, a factor-of-48 increase. In particular, B-tree index builds can
> now benefit from much-larger maintenance_work_mem settings.
This commit causes a bunch of warnings like:
src/backend/utils/sort/tuplesort.c: In function ‘tuplesort_begin_common’:
src/backend/utils/sort/tuplesort.c:399:33: warning: comparison of unsigned expression < 0 is always false
[-Wtype-limits]
#define LACKMEM(state) ((state)->availMem < 0)
to be thrown during compilation. And I think it is spot on. Unless you
overhaul a good bit of the respective logic making availMem unsigned
isn't going to fly.
As is, that code will probably be optimized away which isn't good...
Greetings,
Andres Freund
--
Andres Freund http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2013-07-02 13:47:30 | pgsql: Use an MVCC snapshot, rather than SnapshotNow, for catalog scans |
Previous Message | Peter Eisentraut | 2013-07-02 01:13:42 | pgsql: Fix regression test make dependencies |