dataply 0.0.18-alpha.3 → 0.0.19-alpha.0

package/dist/cjs/index.js

@@ -7941,20 +7941,32 @@ var PageFileSystem = class {
       } else {
         let nextPageId = manager.getNextPageId(page);
         if (nextPageId !== -1) {
-          let pendingFreePageId = nextPageId;
-          while (pendingFreePageId !== -1) {
-            const pendingPage = await this.get(pendingFreePageId, tx);
-            const pendingManager = this.pageFactory.getManager(pendingPage);
-            const next = pendingManager.getNextPageId(pendingPage);
-            await this.setFreePage(pendingFreePageId, tx);
-            pendingFreePageId = next;
-          }
+          await this.freeChain(nextPageId, tx);
           manager.setNextPageId(page, -1);
           await this.setPage(currentPageId, page, tx);
         }
       }
     }
   }
+  /**
+   * Free chain of pages.
+   * @param startPageId Start page ID
+   * @param tx Transaction
+   */
+  async freeChain(startPageId, tx) {
+    let currentPageId = startPageId;
+    const visited = /* @__PURE__ */ new Set();
+    while (currentPageId !== -1 && currentPageId !== 0) {
+      if (visited.has(currentPageId)) {
+        break;
+      }
+      visited.add(currentPageId);
+      const page = await this.get(currentPageId, tx);
+      const nextPageId = this.pageFactory.getManager(page).getNextPageId(page);
+      await this.setFreePage(currentPageId, tx);
+      currentPageId = nextPageId;
+    }
+  }
   /**
    * Frees the page and marks it as available in the bitmap.
    * It also adds the page to the linked list of free pages in metadata.
@@ -7962,6 +7974,7 @@ var PageFileSystem = class {
    * @param tx Transaction
    */
   async setFreePage(pageId, tx) {
+    if (pageId <= 0) return;
     await tx.__acquireWriteLock(0);
     await tx.__acquireWriteLock(pageId);
     const metadata = await this.getMetadata(tx);
@@ -8138,17 +8151,8 @@ var RowIdentifierStrategy = class extends SerializeStrategyAsync {
   }
   async delete(id) {
     const tx = this.txContext.get();
-    const manager = this.factory.getManagerFromType(PageManager.CONSTANT.PAGE_TYPE_INDEX);
-    let pageId = +id;
-    while (true) {
-      const page = await this.pfs.get(pageId, tx);
-      const nextPageId = manager.getNextPageId(page);
-      await this.pfs.setFreePage(pageId, tx);
-      if (nextPageId === -1) {
-        break;
-      }
-      pageId = nextPageId;
-    }
+    const pageId = +id;
+    await this.pfs.freeChain(pageId, tx);
   }
   async readHead() {
     const tx = this.txContext.get();
@@ -8573,14 +8577,8 @@ var RowTableEngine = class {
       return;
     }
     if (this.rowManager.getOverflowFlag(row)) {
-      let overflowPageId = bytesToNumber(this.rowManager.getBody(row));
-      while (overflowPageId !== -1) {
-        const overflowPage = await this.pfs.get(overflowPageId, tx);
-        const manager = this.factory.getManager(overflowPage);
-        const nextPageId = manager.getNextPageId(overflowPage);
-        await this.pfs.setFreePage(overflowPageId, tx);
-        overflowPageId = nextPageId;
-      }
+      const overflowPageId = bytesToNumber(this.rowManager.getBody(row));
+      await this.pfs.freeChain(overflowPageId, tx);
     }
     this.rowManager.setDeletedFlag(row, true);
     await this.pfs.setPage(pageId, page, tx);
@@ -8611,7 +8609,7 @@ var RowTableEngine = class {
       }
     }
     if (allDeleted) {
-      await this.pfs.setFreePage(pageId, tx);
+      await this.pfs.freeChain(pageId, tx);
     }
   }
   /**

package/dist/types/core/PageFileSystem.d.ts

@@ -114,6 +114,12 @@ export declare class PageFileSystem {
      * @param tx Transaction
      */
     writePageContent(pageId: number, data: Uint8Array, offset: number | undefined, tx: Transaction): Promise<void>;
+    /**
+     * Free chain of pages.
+     * @param startPageId Start page ID
+     * @param tx Transaction
+     */
+    freeChain(startPageId: number, tx: Transaction): Promise<void>;
     /**
      * Frees the page and marks it as available in the bitmap.
      * It also adds the page to the linked list of free pages in metadata.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dataply",
-  "version": "0.0.18-alpha.3",
+  "version": "0.0.19-alpha.0",
   "description": "A lightweight storage engine for Node.js with support for MVCC, WAL.",
   "license": "MIT",
   "author": "izure <admin@izure.org>",