@@ -505,10 +505,10 @@ GetOldestUnsummarizedLSN(TimeLineID *tli, bool *lsn_is_exact)
505
505
return InvalidXLogRecPtr ;
506
506
507
507
/*
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.
512
512
*/
513
513
if (!am_wal_summarizer )
514
514
{
@@ -532,8 +532,8 @@ GetOldestUnsummarizedLSN(TimeLineID *tli, bool *lsn_is_exact)
532
532
* Find the oldest timeline on which WAL still exists, and the earliest
533
533
* segment for which it exists.
534
534
*
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
537
537
* under us e.g. if some files were removed, either manually - which
538
538
* shouldn't really happen, but might - or by postgres itself, if
539
539
* summarize_wal was turned off and then back on again.
@@ -582,16 +582,15 @@ GetOldestUnsummarizedLSN(TimeLineID *tli, bool *lsn_is_exact)
582
582
errmsg_internal ("no WAL found on timeline %u" , latest_tli ));
583
583
584
584
/*
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
588
588
* values. Otherwise, we only store values into shared memory if shared
589
589
* 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.
592
591
*/
593
592
LWLockAcquire (WALSummarizerLock , LW_EXCLUSIVE );
594
- if (am_wal_summarizer || !WalSummarizerCtl -> initialized )
593
+ if (am_wal_summarizer || !WalSummarizerCtl -> initialized )
595
594
{
596
595
WalSummarizerCtl -> initialized = true;
597
596
WalSummarizerCtl -> summarized_lsn = unsummarized_lsn ;
0 commit comments