Skip to content

Commit 7ea6231

Browse files
committed
fix(knowledge): size the fill page retry budget to one index maintenance pass
1 parent 01398e0 commit 7ea6231

1 file changed

Lines changed: 6 additions & 5 deletions

File tree

‎packages/db/script-migrations/0021_embedding_search_connector.ts‎

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -23,13 +23,14 @@ export const PROJECTION_SOURCE_ACL_PAGE_PAUSE_MS = 250
2323
export const PROJECTION_SOURCE_ACL_PAGE_TIMEOUT_MS = 60_000
2424

2525
/**
26-
* How many times in a row one page may time out before the run fails. With the pauses below a page
27-
* waits out a few minutes of index maintenance before giving up.
26+
* How many times in a row one page may time out before the run fails. Index maintenance was
27+
* observed holding the page for several minutes; with the pauses below a page waits roughly
28+
* twelve minutes, so the budget covers one such pass.
2829
*/
29-
export const PROJECTION_SOURCE_ACL_PAGE_RETRIES = 6
30+
export const PROJECTION_SOURCE_ACL_PAGE_RETRIES = 12
3031

31-
/** Pause before a page is retried: 10 s, doubling to 30 s, with jitter. */
32-
const PAGE_RETRY_PAUSE = { baseMs: 10_000, maxMs: 30_000 } as const
32+
/** Pause before a page is retried: 10 s, doubling to 60 s, with jitter. */
33+
const PAGE_RETRY_PAUSE = { baseMs: 10_000, maxMs: 60_000 } as const
3334

3435
/**
3536
* The two ways the database cancels a page: `lock_timeout` (55P03) while the page's index write

0 commit comments

Comments
 (0)