From a4448623110c1bcbad54ccd403fb32eba8d50b85 Mon Sep 17 00:00:00 2001
From: Matthew Chen <matthew@signal.org>
Date: Wed, 7 Jul 2021 15:13:30 -0300
Subject: [PATCH] Rework database checkpointing.

---
 .../src/Storage/Database/GRDBDatabaseStorageAdapter.swift   | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/SignalServiceKit/src/Storage/Database/GRDBDatabaseStorageAdapter.swift b/SignalServiceKit/src/Storage/Database/GRDBDatabaseStorageAdapter.swift
index 16938f3b5c..6e49799da0 100644
--- a/SignalServiceKit/src/Storage/Database/GRDBDatabaseStorageAdapter.swift
+++ b/SignalServiceKit/src/Storage/Database/GRDBDatabaseStorageAdapter.swift
@@ -566,10 +566,10 @@ extension GRDBDatabaseStorageAdapter: SDSDatabaseStorageAdapter {
             checkpointBudget -= 1
             return
         }
-        // TODO: We might limit checkpoint frequency by time so that heavy write
-        //       activity won't bog down the main thread.  Ideally, checkpointing
-        //       is fast enough that we don't need to do this.
         var shouldCheckpoint = checkpointBudget <= 0
+
+        // Limit checkpoint frequency by time so that heavy write activity
+        // won't bog down the main thread.
         let maxCheckpointFrequency: TimeInterval = 0.25
         if shouldCheckpoint,
            let lastSuccessfulCheckpointDate = self.lastSuccessfulCheckpointDate,