Re: Re: patch review : Add ability to constrain backend temporary file space - Mailing list pgsql-hackers

From Cédric Villemain
Subject Re: Re: patch review : Add ability to constrain backend temporary file space
Date
Msg-id BANLkTimnryt1sS5Mim_a98aMm=jLRu23Xw@mail.gmail.com
Whole thread Raw
In response to Re: Re: patch review : Add ability to constrain backend temporary file space  (Robert Haas <robertmhaas@gmail.com>)
Responses Re: Re: patch review : Add ability to constrain backend temporary file space
List pgsql-hackers
2011/6/20 Robert Haas <robertmhaas@gmail.com>:
> On Mon, Jun 20, 2011 at 9:15 AM, Cédric Villemain
> <cedric.villemain.debian@gmail.com> wrote:
>> The feature does not work exactly as expected because the write limit
>> is rounded per 8kB because we write before checking. I believe if one
>> write a file of 1GB in one pass (instead of repetitive 8kB increment),
>> and the temp_file_limit is 0, then the server will write the 1GB
>> before aborting.
>
> Can we rearrange thing so we check first, and then write?

probably but it needs more work to catch corner cases. We may be safe
to just document that (and also in the code). The only way I see so
far to have a larger value than 8kB here is to have a plugin doing the
sort instead of the postgresql core sort algo.



>
> --
> Robert Haas
> EnterpriseDB: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>



--
Cédric Villemain               2ndQuadrant
http://2ndQuadrant.fr/     PostgreSQL : Expertise, Formation et Support


pgsql-hackers by date:

Previous
From: Radosław Smogura
Date:
Subject: Re: POSIX question
Next
From: "Kevin Grittner"
Date:
Subject: Re: pika buildfarm member failure on isolationCheck tests