diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2025-01-31 14:36:56 -0500 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2025-01-31 14:36:56 -0500 |
| commit | d4c3a6b8ad830882066122081a7141ecd573f45d (patch) | |
| tree | a007761cdcd4093198d3f2f473da2344e61af261 /src/backend/utils/misc/guc_tables.c | |
| parent | 041e8b95b8cd251bfec6a3c9c3dd6614de6a4c9b (diff) | |
| download | postgresql-d4c3a6b8ad830882066122081a7141ecd573f45d.tar.gz postgresql-d4c3a6b8ad830882066122081a7141ecd573f45d.zip | |
Remove obsolete restriction on the range of log_rotation_size.
When syslogger.c was first written, we didn't want to assume that
all platforms have 64-bit ftello. But we've been assuming that
since v13 (cf commit 799d22461), so let's use that in syslogger.c
and allow log_rotation_size to range up to INT_MAX kilobytes.
The old code effectively limited log_rotation_size to 2GB regardless
of platform. While nobody's complained, that doesn't seem too far
away from what might be thought reasonable these days.
I noticed this while searching for instances of "1024L" in connection
with commit 041e8b95b. These were the last such instances.
(We still have instances of L-suffixed literals, but most of them
are associated with wait intervals for pg_usleep or similar functions.
I don't see any urgent reason to change that.)
Diffstat (limited to 'src/backend/utils/misc/guc_tables.c')
| -rw-r--r-- | src/backend/utils/misc/guc_tables.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/backend/utils/misc/guc_tables.c b/src/backend/utils/misc/guc_tables.c index 38cb9e970d5..0bdfc839078 100644 --- a/src/backend/utils/misc/guc_tables.c +++ b/src/backend/utils/misc/guc_tables.c @@ -3299,7 +3299,7 @@ struct config_int ConfigureNamesInt[] = GUC_UNIT_KB }, &Log_RotationSize, - 10 * 1024, 0, INT_MAX / 1024, + 10 * 1024, 0, INT_MAX, NULL, NULL, NULL }, |