Skip to content

Docs: Enhance maintenance documentation with new actions - #17939

Open
talatuyarer wants to merge 2 commits into
apache:mainfrom
talatuyarer:docs-maintenance-missing-actions
Open

talatuyarer wants to merge 2 commits into
apache:mainfrom
talatuyarer:docs-maintenance-missing-actions

Conversation

@talatuyarer

Copy link
Copy Markdown
Contributor

Adds documentation for six table actions that were missing from maintenance.md

@github-actions github-actions Bot added the docs label Sep 3, 2026

@uros-b uros-b 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.

Every documented action, method signature, default value, option, procedure link, and spec anchor has been traced to upstream/main and verified accurate. Several passages are verbatim the source Javadocs, formatting is clean, and build-docs is green on ubuntu and macOS.

One gap that's maybe worth a follow-up is a single sentence on removeDanglingDeleteFiles's no-op behavior for unpartitioned tables.

  • docs/docs/maintenance.md (removeDanglingDeleteFiles section): RemoveDanglingDeletesSparkAction.execute() short-circuits to an empty result for unpartitioned tables because ManifestFilterManager already cleans dangling deletes on every commit for those tables. Users who call this on an unpartitioned table will see zero files removed and may be confused; one sentence noting the no-op behavior would prevent a common support question.

But overall this is a nice change - thank you @talatuyarer! and please ping relevant committers to take this further, e.g. @aokolnychyi who owns Spark rewrite and compute-stats actions.

@kevinjqliu kevinjqliu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for adding more docs!

I have a few minor comments for clarity and readability.

Please take a look: kevinjqliu@90d16ac

Co-authored-by: Kevin Liu <kevin.jq.liu@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants