Skip to content

feat: add cache-deletion job params schema (MAPCO-11779) - #225

Merged
almog8k merged 1 commit into
alphafrom
fix/MAPCO-11779-cache-deletion-job-params
Sep 23, 2026
Merged

almog8k merged 1 commit into
alphafrom
fix/MAPCO-11779-cache-deletion-job-params

Conversation

@almog8k

@almog8k almog8k commented Sep 22, 2026

Copy link
Copy Markdown
Collaborator
Question Answer
Bug fix ✔
New feature ✔
Breaking change ✖
Deprecations ✖
Documentation ✖
Tests added ✖
Chore ✖

Related issues: MAPCO-11779

Further information:

Adds cacheDeletionJobParamsSchema (zod) and the inferred CacheDeletionJobParams type for Update_Delete_Cache / Swap_Delete_Cache jobs:

  • ingestionJobId: string (uuid)
  • tasksCreationCompleted: boolean

tasksCreationCompleted is set to false by overseer while it is still enqueuing tiles-deletion task batches and flipped to true after the last batch is persisted. job-tracker will gate isJobCompleted() on this flag so it no longer marks the job Completed while later batches are still being enqueued (which currently causes a 409 on POST /jobs/:jobId/tasks and leaves the Redis cache partially deleted).

Both exports are reachable through the existing deletion barrels. Follow-up PRs in overseer and job-tracker will consume the released version.

Add cacheDeletionJobParamsSchema and its inferred CacheDeletionJobParams
type for Update_Delete_Cache / Swap_Delete_Cache jobs. The new
tasksCreationCompleted flag lets job-tracker avoid completing the job
while overseer is still enqueuing task batches.
@github-actions

Copy link
Copy Markdown

Hi it is me, your friendly bot helper! 👋
I've packed this commit for you 😊
You can install it like this:

{
  "dependencies": {
      "@map-colonies/raster-shared": "https://ghatmpstorage.blob.core.windows.net/npm-packages/raster-shared-bd886df0a212a8ef5aae21f8dd2e451ffe623d04.tgz"
  }
}

The link will expire in a week ⌛

@almog8k
almog8k merged commit 720c24f into alpha Sep 23, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants