Re: pgsql: Further thoughts about temp_file_limit patch.
От | Cédric Villemain |
---|---|
Тема | Re: pgsql: Further thoughts about temp_file_limit patch. |
Дата | |
Msg-id | CAF6yO=2nwzvk2OtQr8ubx6jjXG1PQPG_fiX-XLtwBGXeMZB_SA@mail.gmail.com обсуждение исходный текст |
Ответ на | pgsql: Further thoughts about temp_file_limit patch. (Tom Lane <tgl@sss.pgh.pa.us>) |
Ответы |
Re: pgsql: Further thoughts about temp_file_limit patch.
|
Список | pgsql-committers |
2011/7/17 Tom Lane <tgl@sss.pgh.pa.us>: > Further thoughts about temp_file_limit patch. > > Move FileClose's decrement of temporary_files_size up, so that it will be > executed even if elog() throws an error. This is reasonable since if the > unlink() fails, the fact the file is still there is not our fault, and we > are going to forget about it anyhow. So we won't count it against > temp_file_limit anymore. > > Update fileSize and temporary_files_size correctly in FileTruncate. > We probably don't have any places that truncate temp files, but fd.c > surely should not assume that. > > Branch > ------ > master > > Details > ------- > http://git.postgresql.org/pg/commitdiff/9473bb96d0eb7ed73f1bf5269613e6266f64ad85 > > Modified Files > -------------- > src/backend/storage/file/fd.c | 17 +++++++++++++---- > 1 files changed, 13 insertions(+), 4 deletions(-) > > > -- > Sent via pgsql-committers mailing list (pgsql-committers@postgresql.org) > To make changes to your subscription: > http://www.postgresql.org/mailpref/pgsql-committers > How does it sound to remove the stat() call used when log_temp_file is set in favor of fileSize that we now have ? (I didn't check the code rigth now but the 2 patches do not look to touch that part of the code) -- Cédric Villemain 2ndQuadrant http://2ndQuadrant.fr/ PostgreSQL : Expertise, Formation et Support
В списке pgsql-committers по дате отправления: