Re: Re: patch review : Add ability to constrain backend temporary file space

Поиск
Список
Период
Сортировка
Искать
От
Cédric Villemain
Тема
Re: Re: patch review : Add ability to constrain backend temporary file space
Дата
Msg-id
BANLkTimnryt1sS5Mim_a98aMm=jLRu23Xw@mail.gmail.com
Ответ на
Список
Дерево обсуждения
patch review : Add ability to constrain backend temporary file space Cédric Villemain <cedric.villemain.debian@gmail.com>
Re: patch review : Add ability to constrain backend temporary file space Mark Kirkwood <mark.kirkwood@catalyst.net.nz>
Re: patch review : Add ability to constrain backend temporary file space Mark Kirkwood <mark.kirkwood@catalyst.net.nz>
Re: patch review : Add ability to constrain backend temporary file space Mark Kirkwood <mark.kirkwood@catalyst.net.nz>
Re: patch review : Add ability to constrain backend temporary file space Mark Kirkwood <mark.kirkwood@catalyst.net.nz>
Re: Re: patch review : Add ability to constrain backend temporary file space Mark Kirkwood <mark.kirkwood@catalyst.net.nz>
Re: Re: patch review : Add ability to constrain backend temporary file space Robert Haas <robertmhaas@gmail.com>
Re: Re: patch review : Add ability to constrain backend temporary file space Tom Lane <tgl@sss.pgh.pa.us>
Re: Re: patch review : Add ability to constrain backend temporary file space Mark Kirkwood <mark.kirkwood@catalyst.net.nz>
Re: patch review : Add ability to constrain backend temporary file space Greg Stark <gsstark@mit.edu>
Re: Re: patch review : Add ability to constrain backend temporary file space Cédric Villemain <cedric.villemain.debian@gmail.com>
Re: Re: patch review : Add ability to constrain backend temporary file space Robert Haas <robertmhaas@gmail.com>
Re: Re: patch review : Add ability to constrain backend temporary file space Jaime Casanova <jaime@2ndquadrant.com>
Re: Re: patch review : Add ability to constrain backend temporary file space Mark Kirkwood <mark.kirkwood@catalyst.net.nz>
Re: Re: patch review : Add ability to constrain backend temporary file space Mark Kirkwood <mark.kirkwood@catalyst.net.nz>
Re: Re: patch review : Add ability to constrain backend temporary file space Cédric Villemain <cedric.villemain.debian@gmail.com>
Re: Re: patch review : Add ability to constrain backend temporary file space Mark Kirkwood <mark.kirkwood@catalyst.net.nz>
Re: Re: patch review : Add ability to constrain backend temporary file space Cédric Villemain <cedric.villemain.debian@gmail.com>
Re: Re: patch review : Add ability to constrain backend temporary file space Mark Kirkwood <mark.kirkwood@catalyst.net.nz>
Re: Re: patch review : Add ability to constrain backend temporary file space Mark Kirkwood <mark.kirkwood@catalyst.net.nz>
Re: Re: patch review : Add ability to constrain backend temporary file space Cédric Villemain <cedric.villemain.debian@gmail.com>
Re: Re: patch review : Add ability to constrain backend temporary file space Cédric Villemain <cedric.villemain.debian@gmail.com>
Re: Re: patch review : Add ability to constrain backend temporary file space Robert Haas <robertmhaas@gmail.com>
Re: Re: patch review : Add ability to constrain backend temporary file space Cédric Villemain <cedric.villemain.debian@gmail.com>
Re: Re: patch review : Add ability to constrain backend temporary file space Mark Kirkwood <mark.kirkwood@catalyst.net.nz>
Re: Re: patch review : Add ability to constrain backend temporary file space Mark Kirkwood <mark.kirkwood@catalyst.net.nz>
Re: Re: patch review : Add ability to constrain backend temporary file space Tom Lane <tgl@sss.pgh.pa.us>
Re: Re: patch review : Add ability to constrain backend temporary file space Mark Kirkwood <mark.kirkwood@catalyst.net.nz>
Re: Re: patch review : Add ability to constrain backend temporary file space Mark Kirkwood <mark.kirkwood@catalyst.net.nz>
Re: Re: patch review : Add ability to constrain backend temporary file space Tom Lane <tgl@sss.pgh.pa.us>
Re: Re: patch review : Add ability to constrain backend temporary file space Cédric Villemain <cedric.villemain.debian@gmail.com>
Re: Re: patch review : Add ability to constrain backend temporary file space Josh Berkus <josh@agliodbs.com>
Re: Re: patch review : Add ability to constrain backend temporary file space Tatsuo Ishii <ishii@postgresql.org>
Re: Re: patch review : Add ability to constrain backend temporary file space Mark Kirkwood <mark.kirkwood@catalyst.net.nz>
Re: Re: patch review : Add ability to constrain backend temporary file space Tatsuo Ishii <ishii@postgresql.org>
Re: Re: patch review : Add ability to constrain backend temporary file space Mark Kirkwood <mark.kirkwood@catalyst.net.nz>
Re: Re: patch review : Add ability to constrain backend temporary file space Cédric Villemain <cedric.villemain.debian@gmail.com>
Re: Re: patch review : Add ability to constrain backend temporary file space Tatsuo Ishii <ishii@postgresql.org>
Re: Re: patch review : Add ability to constrain backend temporary file space Tom Lane <tgl@sss.pgh.pa.us>
Re: Re: patch review : Add ability to constrain backend temporary file space Tatsuo Ishii <ishii@postgresql.org>
Re: Re: patch review : Add ability to constrain backend temporary file space Tom Lane <tgl@sss.pgh.pa.us>
Re: Re: patch review : Add ability to constrain backend temporary file space Tatsuo Ishii <ishii@postgresql.org>
Re: Re: patch review : Add ability to constrain backend temporary file space Tom Lane <tgl@sss.pgh.pa.us>
Re: patch review : Add ability to constrain backend temporary file space Cédric Villemain <cedric.villemain.debian@gmail.com>
Re: patch review : Add ability to constrain backend temporary file space Mark Kirkwood <mark.kirkwood@catalyst.net.nz>
2011/6/20 Robert Haas :
> On Mon, Jun 20, 2011 at 9:15 AM, Cédric Villemain
>  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 по дате отправления
От: Radosław Smogura
Дата:
Сообщение: Re: POSIX question
От: Kevin Grittner
Дата:
FAQ