From 4c999fb6512be242b99fcaecb6dffa1fb018e153 Mon Sep 17 00:00:00 2001 From: Benjamin Schmidt Date: Tue, 15 Jul 2025 04:23:10 -0400 Subject: [PATCH] chore: fix cleanupOlderThan docs (#2504) Thanks for all your work. The docstring for `OptimizeOptions ` seems to reference a non-existent method on `Table`. I believe this is the correct example for `cleanupOlderThan`. This also appears in the generated docs, but I assume they live downstream from this code? --- nodejs/lancedb/table.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/nodejs/lancedb/table.ts b/nodejs/lancedb/table.ts index e344a7f5..8c62a927 100644 --- a/nodejs/lancedb/table.ts +++ b/nodejs/lancedb/table.ts @@ -75,10 +75,10 @@ export interface OptimizeOptions { * // Delete all versions older than 1 day * const olderThan = new Date(); * olderThan.setDate(olderThan.getDate() - 1)); - * tbl.cleanupOlderVersions(olderThan); + * tbl.optimize({cleanupOlderThan: olderThan}); * * // Delete all versions except the current version - * tbl.cleanupOlderVersions(new Date()); + * tbl.optimize({cleanupOlderThan: new Date()}); */ cleanupOlderThan: Date; deleteUnverified: boolean;