@@ -505,10 +505,10 @@ GetOldestUnsummarizedLSN(TimeLineID *tli, bool *lsn_is_exact)
505505return InvalidXLogRecPtr ;
506506
507507/*
508- * If we are not the WAL summarizer process, then we normally just want
509- *to read the values from shared memory. However, as an exception, if
510- *shared memory hasn't been initialized yet, then we need to do that so
511- *that we can read legal values and not remove any WAL too early.
508+ * If we are not the WAL summarizer process, then we normally just want to
509+ * read the values from shared memory. However, as an exception, if shared
510+ * memory hasn't been initialized yet, then we need to do that so that we
511+ * can read legal values and not remove any WAL too early.
512512 */
513513if (!am_wal_summarizer )
514514{
@@ -532,8 +532,8 @@ GetOldestUnsummarizedLSN(TimeLineID *tli, bool *lsn_is_exact)
532532 * Find the oldest timeline on which WAL still exists, and the earliest
533533 * segment for which it exists.
534534 *
535- * Note that we do this every time the WAL summarizer process restarts
536- *or recovers from an error, in case the contents of pg_wal have changed
535+ * Note that we do this every time the WAL summarizer process restarts or
536+ * recovers from an error, in case the contents of pg_wal have changed
537537 * under us e.g. if some files were removed, either manually - which
538538 * shouldn't really happen, but might - or by postgres itself, if
539539 * summarize_wal was turned off and then back on again.
@@ -582,16 +582,15 @@ GetOldestUnsummarizedLSN(TimeLineID *tli, bool *lsn_is_exact)
582582errmsg_internal ("no WAL found on timeline %u" ,latest_tli ));
583583
584584/*
585- * If we're the WAL summarizer, we always want to store the values we
586- *just computed into shared memory, because those are the values we're
587- *going to use to drive our operation, and so they are the authoritative
585+ * If we're the WAL summarizer, we always want to store the values we just
586+ * computed into shared memory, because those are the values we're going
587+ * to use to drive our operation, and so they are the authoritative
588588 * values. Otherwise, we only store values into shared memory if shared
589589 * memory is uninitialized. Our values are not canonical in such a case,
590- * but it's better to have something than nothing, to guide WAL
591- * retention.
590+ * but it's better to have something than nothing, to guide WAL retention.
592591 */
593592LWLockAcquire (WALSummarizerLock ,LW_EXCLUSIVE );
594- if (am_wal_summarizer || !WalSummarizerCtl -> initialized )
593+ if (am_wal_summarizer || !WalSummarizerCtl -> initialized )
595594{
596595WalSummarizerCtl -> initialized = true;
597596WalSummarizerCtl -> summarized_lsn = unsummarized_lsn ;