diff options
author | Noah Misch <noah@leadboat.com> | 2013-07-04 23:09:54 -0400 |
---|---|---|
committer | Noah Misch <noah@leadboat.com> | 2013-07-04 23:13:54 -0400 |
commit | 79e0f87a15643efa9a94e011da509746dbb96798 (patch) | |
tree | 35a5014eddaafda2d63158ea92bf788a6a0a52ca /src/backend/utils/adt/pgstatfuncs.c | |
parent | 7842d41df56ab8ad9aff76bc5bd8e96c3d215cde (diff) | |
download | postgresql-79e0f87a15643efa9a94e011da509746dbb96798.tar.gz postgresql-79e0f87a15643efa9a94e011da509746dbb96798.zip |
Use type "int64" for memory accounting in tuplesort.c/tuplestore.c.
Commit 263865a48973767ce8ed7b7788059a38a24a9f37 switched tuplesort.c and
tuplestore.c variables representing memory usage from type "long" to
type "Size". This was unnecessary; I thought doing so avoided overflow
scenarios on 64-bit Windows, but guc.c already limited work_mem so as to
prevent the overflow. It was also incomplete, not touching the logic
that assumed a signed data type. Change the affected variables to
"int64". This is perfect for 64-bit platforms, and it reduces the need
to contemplate platform-specific overflow scenarios. It also puts us
close to being able to support work_mem over 2 GiB on 64-bit Windows.
Per report from Andres Freund.
Diffstat (limited to 'src/backend/utils/adt/pgstatfuncs.c')
0 files changed, 0 insertions, 0 deletions