Skip to content

feat(operation): introduce cleanup operations#128

Open
zjw1111 wants to merge 1 commit into
apache:mainfrom
zjw1111:migrate/orphan-expire-cleanup
Open

feat(operation): introduce cleanup operations#128
zjw1111 wants to merge 1 commit into
apache:mainfrom
zjw1111:migrate/orphan-expire-cleanup

Conversation

@zjw1111

@zjw1111 zjw1111 commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Purpose

Linked issue: N/A

Introduce snapshot expiration and orphan file cleanup operation components migrated from the Alibaba Paimon C++ repository.

Migrated files:

  • include/paimon/orphan_files_cleaner.h
  • src/paimon/core/operation/commit_metrics_test.cpp
  • src/paimon/core/operation/expire_snapshots.cpp
  • src/paimon/core/operation/expire_snapshots.h
  • src/paimon/core/operation/expire_snapshots_test.cpp
  • src/paimon/core/operation/orphan_files_cleaner.cpp
  • src/paimon/core/operation/orphan_files_cleaner_impl.cpp
  • src/paimon/core/operation/orphan_files_cleaner_impl.h
  • src/paimon/core/operation/orphan_files_cleaner_test.cpp
  • src/paimon/core/operation/metrics/clean_metrics.h

No extra dependency files were migrated in this pull request.

License handling:

  • Replaced Alibaba copyright headers with ASF license headers.
  • No third-party license or notice updates were required for this batch.

External contributor handling:

  • No Co-authored-by trailer was added.
  • chongchongxiao <37289259+chongchongxiao@users.noreply.github.com> exceeded the per-file contribution threshold and will be acknowledged in a pull request comment.

Tests

API and Format

This change adds the public OrphanFilesCleaner API in include/paimon/orphan_files_cleaner.h.

Documentation

No documentation changes.

Generative AI tooling

Migrated-by: OpenAI Codex

Copilot AI review requested due to automatic review settings June 25, 2026 07:51

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@zjw1111

zjw1111 commented Jun 25, 2026

Copy link
Copy Markdown
Contributor Author

Thanks @chongchongxiao for your previous contributions to paimon-cpp. This migration PR carries that work forward into Apache Paimon C++.

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