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 по дате отправления
От: Kevin Grittner
Дата: