From bfc27bee5e3e8c1df1d904dd80969917314e02e2 Mon Sep 17 00:00:00 2001 From: Heikki Linnakangas Date: Sat, 10 Jul 2021 18:53:47 +0300 Subject: [PATCH] Revert the fix to allegedly inaccurate comment. I misread the code. It does indeed only call checkpoint() every 10 segments. Revert that change, but keep the rest of the comment fixes. --- pageserver/src/walreceiver.rs | 2 +- vendor/postgres | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/pageserver/src/walreceiver.rs b/pageserver/src/walreceiver.rs index 420eecaeff..8885446b2e 100644 --- a/pageserver/src/walreceiver.rs +++ b/pageserver/src/walreceiver.rs @@ -198,7 +198,7 @@ fn walreceiver_main( // flush ptr. timeline.advance_last_valid_lsn(endlsn); - // Somewhat arbitrarily, if we have at least one complete wal segment (16 MB each), + // Somewhat arbitrarily, if we have at least 10 complete wal segments (16 MB each), // "checkpoint" the repository to flush all the changes from WAL we've processed // so far to disk. After this, we don't need the original WAL anymore, and it // can be removed. This is probably too aggressive for production, but it's useful diff --git a/vendor/postgres b/vendor/postgres index 8478beabaf..a08f50cba2 160000 --- a/vendor/postgres +++ b/vendor/postgres @@ -1 +1 @@ -Subproject commit 8478beabaf7d6916d22bde99ee5a577a9b6b237c +Subproject commit a08f50cba293fff045cb771f8c6bb23b89f4f7f7