From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | pgsql: In AtEOXact_Files, complain if any files remain unclosed at comm |
Date: | 2018-04-28 21:45:09 |
Message-ID: | E1fCXeX-00040u-2d@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
In AtEOXact_Files, complain if any files remain unclosed at commit.
This change makes this module act more like most of our other low-level
resource management modules. It's a caller error if something is not
explicitly closed by the end of a successful transaction, so issue
a WARNING about it. This would not actually have caught the file leak
bug fixed in commit 231bcd080, because that was in a transaction-abort
path; but it still seems like a good, and pretty cheap, cross-check.
Discussion: https://postgr.es/m/152056616579.4966.583293218357089052@wrigleys.postgresql.org
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/9cb7db3f0c1f554cdcbbd97f78a119a19756e6ef
Modified Files
--------------
src/backend/access/transam/xact.c | 6 +++---
src/backend/postmaster/autovacuum.c | 2 +-
src/backend/postmaster/bgwriter.c | 2 +-
src/backend/postmaster/checkpointer.c | 2 +-
src/backend/postmaster/walwriter.c | 2 +-
src/backend/storage/file/fd.c | 32 +++++++++++++++++++++-----------
src/include/storage/fd.h | 2 +-
7 files changed, 29 insertions(+), 19 deletions(-)
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2018-04-29 17:26:35 | pgsql: Cosmetic improvement: use BKI_DEFAULT and BKI_LOOKUP in pg_langu |
Previous Message | Tom Lane | 2018-04-28 20:46:27 | pgsql: Fix incorrect field type for PlannedStmt.jitFlags in outfuncs/re |