Thread

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →
  1. Add temp_file_limit GUC parameter to constrain temporary file space usage.

  1. pgsql: Add temp_file_limit GUC parameter to constrain temporary file sp

    Tom Lane <tgl@sss.pgh.pa.us> — 2011-07-17T18:19:51Z

    Add temp_file_limit GUC parameter to constrain temporary file space usage.
    
    The limit is enforced against the total amount of temp file space used by
    each session.
    
    Mark Kirkwood, reviewed by Cédric Villemain and Tatsuo Ishii
    
    Branch
    ------
    master
    
    Details
    -------
    http://git.postgresql.org/pg/commitdiff/23e5b16c71f2706978c5f54aabd28ed23a54b6a5
    
    Modified Files
    --------------
    doc/src/sgml/config.sgml                      |   33 +++++++++++-
    src/backend/storage/file/fd.c                 |   69 ++++++++++++++++++++++---
    src/backend/utils/errcodes.txt                |   11 ++--
    src/backend/utils/misc/guc.c                  |   15 +++++
    src/backend/utils/misc/postgresql.conf.sample |    5 ++
    src/include/utils/guc.h                       |    2 +
    src/include/utils/guc_tables.h                |    1 +
    7 files changed, 122 insertions(+), 14 deletions(-)
    
    
  2. Re: [COMMITTERS] pgsql: Add temp_file_limit GUC parameter to constrain temporary file sp

    Bruce Momjian <bruce@momjian.us> — 2011-07-17T21:08:56Z

    Tom Lane wrote:
    > Add temp_file_limit GUC parameter to constrain temporary file space usage.
    > 
    > The limit is enforced against the total amount of temp file space used by
    > each session.
    > 
    > Mark Kirkwood, reviewed by C?dric Villemain and Tatsuo Ishii
    
    Should we document that sessions that exceed this limit generate an
    error?  I don't see any mention of this in the docs.
    
    -- 
      Bruce Momjian  <bruce@momjian.us>        http://momjian.us
      EnterpriseDB                             http://enterprisedb.com
    
      + It's impossible for everything to be true. +
    
    
  3. Re: [COMMITTERS] pgsql: Add temp_file_limit GUC parameter to constrain temporary file sp

    Tom Lane <tgl@sss.pgh.pa.us> — 2011-07-17T21:10:02Z

    Bruce Momjian <bruce@momjian.us> writes:
    > Tom Lane wrote:
    >> Add temp_file_limit GUC parameter to constrain temporary file space usage.
    >> 
    >> The limit is enforced against the total amount of temp file space used by
    >> each session.
    >> 
    >> Mark Kirkwood, reviewed by C?dric Villemain and Tatsuo Ishii
    
    > Should we document that sessions that exceed this limit generate an
    > error?  I don't see any mention of this in the docs.
    
    Huh?  Seems like a waste of text to me.  What else would happen?
    
    			regards, tom lane
    
    
  4. Re: [COMMITTERS] pgsql: Add temp_file_limit GUC parameter to constrain temporary file sp

    Bruce Momjian <bruce@momjian.us> — 2011-07-17T22:39:19Z

    Tom Lane wrote:
    > Bruce Momjian <bruce@momjian.us> writes:
    > > Tom Lane wrote:
    > >> Add temp_file_limit GUC parameter to constrain temporary file space usage.
    > >> 
    > >> The limit is enforced against the total amount of temp file space used by
    > >> each session.
    > >> 
    > >> Mark Kirkwood, reviewed by C?dric Villemain and Tatsuo Ishii
    > 
    > > Should we document that sessions that exceed this limit generate an
    > > error?  I don't see any mention of this in the docs.
    > 
    > Huh?  Seems like a waste of text to me.  What else would happen?
    
    Well, if we exceed work_mem, we spill to temp disk.  If we exceed temp
    disk, we error out.  Those different behaviors don't seem obvious to me.
    The work_mem docs do mention is spills to disk though, so maybe it is
    OK.
    
    -- 
      Bruce Momjian  <bruce@momjian.us>        http://momjian.us
      EnterpriseDB                             http://enterprisedb.com
    
      + It's impossible for everything to be true. +
    
    
  5. Re: Re: [COMMITTERS] pgsql: Add temp_file_limit GUC parameter to constrain temporary file sp

    Peter Eisentraut <peter_e@gmx.net> — 2011-07-18T10:52:50Z

    On sön, 2011-07-17 at 18:39 -0400, Bruce Momjian wrote:
    > Tom Lane wrote:
    > > Bruce Momjian <bruce@momjian.us> writes:
    > > > Tom Lane wrote:
    > > >> Add temp_file_limit GUC parameter to constrain temporary file space usage.
    > > >> 
    > > >> The limit is enforced against the total amount of temp file space used by
    > > >> each session.
    > > >> 
    > > >> Mark Kirkwood, reviewed by C?dric Villemain and Tatsuo Ishii
    > > 
    > > > Should we document that sessions that exceed this limit generate an
    > > > error?  I don't see any mention of this in the docs.
    > > 
    > > Huh?  Seems like a waste of text to me.  What else would happen?
    > 
    > Well, if we exceed work_mem, we spill to temp disk.  If we exceed temp
    > disk, we error out.  Those different behaviors don't seem obvious to me.
    > The work_mem docs do mention is spills to disk though, so maybe it is
    > OK.
    
    Sounds like it would be good to document the behavior if the limit is
    exceeded in each case.
    
    
    
  6. Re: Re: [COMMITTERS] pgsql: Add temp_file_limit GUC parameter to constrain temporary file sp

    Tom Lane <tgl@sss.pgh.pa.us> — 2011-07-18T14:36:07Z

    Bruce Momjian <bruce@momjian.us> writes:
    > Tom Lane wrote:
    >> Huh?  Seems like a waste of text to me.  What else would happen?
    
    > Well, if we exceed work_mem, we spill to temp disk.  If we exceed temp
    > disk, we error out.  Those different behaviors don't seem obvious to me.
    
    [ shrug... ]  Feel free to change it.
    
    			regards, tom lane
    
    
  7. Re: Re: [COMMITTERS] pgsql: Add temp_file_limit GUC parameter to constrain temporary file sp

    Mark Kirkwood <mark.kirkwood@catalyst.net.nz> — 2011-07-18T22:29:45Z

    On 19/07/11 02:36, Tom Lane wrote:
    > Bruce Momjian<bruce@momjian.us>  writes:
    >> Tom Lane wrote:
    >>> Huh?  Seems like a waste of text to me.  What else would happen?
    >> Well, if we exceed work_mem, we spill to temp disk.  If we exceed temp
    >> disk, we error out.  Those different behaviors don't seem obvious to me.
    > [ shrug... ]  Feel free to change it.
    >
    >
    
    No objections from me - can't see any harm in making it very clear what 
    happens when the limit is exceeded :-)
    
    
    
  8. Re: Re: [COMMITTERS] pgsql: Add temp_file_limit GUC parameter to constrain temporary file sp

    Bruce Momjian <bruce@momjian.us> — 2012-08-03T19:15:56Z

    On Tue, Jul 19, 2011 at 10:29:45AM +1200, Mark Kirkwood wrote:
    > On 19/07/11 02:36, Tom Lane wrote:
    > >Bruce Momjian<bruce@momjian.us>  writes:
    > >>Tom Lane wrote:
    > >>>Huh?  Seems like a waste of text to me.  What else would happen?
    > >>Well, if we exceed work_mem, we spill to temp disk.  If we exceed temp
    > >>disk, we error out.  Those different behaviors don't seem obvious to me.
    > >[ shrug... ]  Feel free to change it.
    > >
    > >
    > 
    > No objections from me - can't see any harm in making it very clear
    > what happens when the limit is exceeded :-)
    
    Documentation patch attached and applied, and backpatched to 9.2.
    
    -- 
      Bruce Momjian  <bruce@momjian.us>        http://momjian.us
      EnterpriseDB                             http://enterprisedb.com
    
      + It's impossible for everything to be true. +