diff options
author | Thomas Munro <tmunro@postgresql.org> | 2023-05-15 10:45:19 +1200 |
---|---|---|
committer | Thomas Munro <tmunro@postgresql.org> | 2023-05-15 11:19:54 +1200 |
commit | 63932a6d38e5dfa6df2a51a04b7314ec1e4d3de7 (patch) | |
tree | 38bf9c3cf2a270d717e198f4f5db8ffd77808078 | |
parent | 319bae9a8da66678d80d862440e54cebc7b1cc24 (diff) | |
download | postgresql-63932a6d38e5dfa6df2a51a04b7314ec1e4d3de7.tar.gz postgresql-63932a6d38e5dfa6df2a51a04b7314ec1e4d3de7.zip |
Fix wal_writer_flush_after initializer value.
Commit a73952b7956 (new in 16) required default values in guc_table.c
and C variable initializers to match. This one only matched when
XLOG_BLCKSZ == 8kB. Fix by using the same expression in both places
with a new DEFAULT_XXX macro, as done for other GUCs.
Reviewed-by: Andres Freund <andres@anarazel.de>
Discussion: https://postgr.es/m/CA+hUKGLNmLV=VrT==5MqnbARgx2ifRSFtdd8ofdfrdSLL3yv5A@mail.gmail.com
-rw-r--r-- | src/backend/postmaster/walwriter.c | 2 | ||||
-rw-r--r-- | src/backend/utils/misc/guc_tables.c | 2 | ||||
-rw-r--r-- | src/include/postmaster/walwriter.h | 2 |
3 files changed, 4 insertions, 2 deletions
diff --git a/src/backend/postmaster/walwriter.c b/src/backend/postmaster/walwriter.c index 65e84be39b9..266fbc23399 100644 --- a/src/backend/postmaster/walwriter.c +++ b/src/backend/postmaster/walwriter.c @@ -68,7 +68,7 @@ * GUC parameters */ int WalWriterDelay = 200; -int WalWriterFlushAfter = 128; +int WalWriterFlushAfter = DEFAULT_WAL_WRITER_FLUSH_AFTER; /* * Number of do-nothing loops before lengthening the delay time, and the diff --git a/src/backend/utils/misc/guc_tables.c b/src/backend/utils/misc/guc_tables.c index efd59a47cf7..8abf9bb6446 100644 --- a/src/backend/utils/misc/guc_tables.c +++ b/src/backend/utils/misc/guc_tables.c @@ -2780,7 +2780,7 @@ struct config_int ConfigureNamesInt[] = GUC_UNIT_XBLOCKS }, &WalWriterFlushAfter, - (1024 * 1024) / XLOG_BLCKSZ, 0, INT_MAX, + DEFAULT_WAL_WRITER_FLUSH_AFTER, 0, INT_MAX, NULL, NULL, NULL }, diff --git a/src/include/postmaster/walwriter.h b/src/include/postmaster/walwriter.h index 22281a97ba0..6eba7ad79cf 100644 --- a/src/include/postmaster/walwriter.h +++ b/src/include/postmaster/walwriter.h @@ -12,6 +12,8 @@ #ifndef _WALWRITER_H #define _WALWRITER_H +#define DEFAULT_WAL_WRITER_FLUSH_AFTER ((1024 * 1024) / XLOG_BLCKSZ) + /* GUC options */ extern PGDLLIMPORT int WalWriterDelay; extern PGDLLIMPORT int WalWriterFlushAfter; |