Skip to content

feat(global-index): add checkpoint cleanup by task id - #465

Merged
lxy-9602 merged 2 commits into
apache:mainfrom
lszskye:checkpoint_clean_api
Oct 27, 2026
Merged

lxy-9602 merged 2 commits into
apache:mainfrom
lszskye:checkpoint_clean_api

Conversation

@lszskye

@lszskye lszskye commented Oct 26, 2026

Copy link
Copy Markdown
Collaborator

Purpose

Global index builds remove checkpoints after successful completion, but a process crash can leave files behind. Add GlobalIndexWriteTask::DeleteCheckpoint to recursively remove all leftovers for a task once it is no longer running or being retried.

Linked issue: #382

Tests

  • FileStorePathFactoryTest.* and GlobalIndexFileManagerTest.*:
  • GlobalIndexTest.TestWriteLuminaIndexWithCheckpoint
  • Lumina checkpoint and tag-filter tests

API and Format

Add the public API:

GlobalIndexWriteTask::DeleteCheckpoint(table_path, task_id, options, file_system)

Task IDs must be unique within a table.

Documentation

Add checkpoint cleanup usage and task ID requirements to the global index user guide.

Generative AI tooling

Generated-by: OpenAI Codex (GPT-6)

@lxy-9602 lxy-9602 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@lxy-9602
lxy-9602 merged commit a774f8e into apache:main Oct 27, 2026
16 checks passed
lucasfang pushed a commit to lucasfang/paimon-cpp that referenced this pull request Oct 27, 2026
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