Re: current_logfiles not following group access and instead follows log_file_mode permissions
Michael Paquier <michael@paquier.xyz>
From: Michael Paquier <michael@paquier.xyz>
To: Stephen Frost <sfrost@snowman.net>
Cc: Haribabu Kommi <kommi.haribabu@gmail.com>, Tom Lane <tgl@sss.pgh.pa.us>, pgsql-hackers@lists.postgresql.org, Gilles Darold <gilles.darold@dalibo.com>
Date: 2019-01-19T01:49:14Z
Lists: pgsql-hackers
On Fri, Jan 18, 2019 at 09:50:40AM -0500, Stephen Frost wrote: > It'd probably be good to give folks an opportunity to voice their > opinion regarding their use-case for the file existing as it does and > being documented as it is. At first blush, to me anyway, it seems like > maybe this was a case of "over-documenting" of the feature by including > in user-facing documentation something that was really there for > internal reasons, but I could certainly be wrong and maybe there's a > reason why it's really necessary to have the file around for users. It's not only that. By keeping the file in its current location, we can prevent base backups to work even if logs files are out of the data folder, which is rather user-friendly, and I think that advanced users of Postgres are careful enough to split log files and main data folders into different partitions, without symlinks from the data folder to the log location and with log_directory set to an absolute path, independent of the rest. So moving current_logfiles out of the data folder to the base location of the log paths makes quite some sense in my opinion for consistency. Using a new GUC to specify where current_logfiles should be located does not really justify the code complications in my opinion, and I'd think that we should allow users with log file access to still look at it, even manually and connected from the host as this can be useful for debugging purposes (sometimes clocks of systems get changed as they are not all the time going throuhg ntpd). -- Michael
Commits
-
Make current_logfiles use permissions assigned to files in data directory
- 7d7435c5c505 11.3 landed
- 276d2e6c2d81 12.0 landed