Attach the kj::Path in alarm-scheduler.c++

This is an attempt to fix #1386
cloudflare · Nov 27, 2023 · a10ceed · a10ceed
1 parent b230f0f
commit a10ceed
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/src/workerd/server/alarm-scheduler.c++ b/src/workerd/server/alarm-scheduler.c++
@@ -28,11 +28,12 @@ AlarmScheduler::AlarmScheduler(
     const kj::Clock& clock,
     kj::Timer& timer,
     const SqliteDatabase::Vfs& vfs,
-    kj::PathPtr path)
+    kj::Path path)
     : clock(clock), timer(timer), random(makeSeededRandomEngine()),
       db([&]{
         auto db = kj::heap<SqliteDatabase>(vfs, path,
-            kj::WriteMode::CREATE | kj::WriteMode::MODIFY | kj::WriteMode::CREATE_PARENT);
+            kj::WriteMode::CREATE | kj::WriteMode::MODIFY | kj::WriteMode::CREATE_PARENT)
+                .attach(kj::mv(path));
         ensureInitialized(*db);
         return kj::mv(db);
       }()),

diff --git a/src/workerd/server/alarm-scheduler.h b/src/workerd/server/alarm-scheduler.h
@@ -64,7 +64,7 @@ class AlarmScheduler final : kj::TaskSet::ErrorHandler {
     const kj::Clock& clock,
     kj::Timer& timer,
     const SqliteDatabase::Vfs& vfs,
-    kj::PathPtr path);
+    kj::Path path);
 
   kj::Maybe<kj::Date> getAlarm(ActorKey actor);
   bool setAlarm(ActorKey actor, kj::Date scheduledTime);