Docs: Add docs for Table Maintenance in Flink by JeonDaehong · Pull Request #13853 · apache/iceberg

JeonDaehong · 2025-08-18T13:11:56Z

This PR adds documentation for issue
( Docs update for Flink Table Maintenance API - Fixes #13307 )

It incorporates feedback received from the previously closed PR #13360. and #13771

Please review and let me know if any changes are needed. I will update promptly.

JeonDaehong · 2025-08-18T13:14:09Z

Hello,

I have submitted a new PR with the previous feedback incorporated as much as possible.

Since this is my first time contributing to the Docs, I am still learning and going through some trial and error. I really appreciate all your guidance and support.

I will gladly incorporate any additional feedback you may have.

If everything looks good, I would appreciate it if you could merge the PR.

Thank you very much.

pvary · 2025-08-18T13:48:50Z

+
+## Overview
+
+In **Apache Iceberg** deployments within **Flink streaming environments**, implementing automated table maintenance operations—including `snapshot expiration`, `small file compaction`, and `orphan file cleanup`—is critical for optimal performance and storage efficiency.


optimal performance -> optimal query performance

Let's see if we can merge the orphan file cleanup in 1.10.0.

If yes, then we need to mentioned it in the features

If no, then we should remove it from here

pvary · 2025-08-18T13:53:41Z

+
+Example usage:
+```java
+// Snapshot expiration
+.add(ExpireSnapshots.builder()
+    .maxSnapshotAge(Duration.ofDays(7))
+    .retainLast(10)
+    .deleteBatchSize(1000))
+
+// Continuous file optimization
+.add(RewriteDataFiles.builder()
+    .targetFileSizeBytes(256 * 1024 * 1024)
+    .minFileSizeBytes(32 * 1024 * 1024)
+    .partialProgressEnabled(true)
+    .partialProgressMaxCommits(5))
+```


Do we need this example here? We have more detailed examples below.
If we decide to need this, we might want to separate the ExpireSnapshots and the RewriteDataFiles and put them after the descriptions above

JeonDaehong · 2025-08-19T12:46:24Z

@pvary

I have incorporated the feedback you provided.

I would appreciate it if you could review it.

Where can I check the addition of orphan file cleanup in version 1.10.0?

JeonDaehong · 2025-08-19T12:47:25Z

@pvary

Adding to the docs is more challenging than I expected, but also really fun.
Still, it’s enjoyable and rewarding.

I hope it gets merged soon :D

Guosmilesmile · 2025-08-19T13:26:55Z

+```java
+TriggerLockFactory lockFactory = new ZkLockFactory(
+    "localhost:2181",        // ZooKeeper connection string
+    "catalog.db.table"       // Lock ID (unique identifier)


We also need sessionTimeoutMs,connectionTimeoutMs,baseSleepTimeMs and maxRetries in Constructor.

Guosmilesmile · 2025-08-19T13:30:28Z

+
+| Method | Description | Default Value | Type |
+|--------|-------------|---------------|------|
+| `maxSnapshotAge(Duration)` | Maximum age of snapshots to retain | None | Duration |


Default value is 5 days.

Guosmilesmile · 2025-08-19T13:31:59Z

+| Method | Description | Default Value | Type |
+|--------|-------------|---------------|------|
+| `maxSnapshotAge(Duration)` | Maximum age of snapshots to retain | None | Duration |
+| `retainLast(int)` | Minimum number of snapshots to retain | None | int |


Default value is 1.

Guosmilesmile · 2025-08-19T13:33:27Z

+| `retainLast(int)` | Minimum number of snapshots to retain | None | int |
+| `deleteBatchSize(int)` | Number of files to delete in each batch | 1000 | int |
+| `planningWorkerPoolSize(int)` | Number of worker threads for planning snapshot expiration | Shared worker pool | int |
+| `cleanExpiredMetadata(boolean)` | Remove expired metadata files when expiring snapshots | true | boolean |


I think in this place default value is false.

Guosmilesmile · 2025-08-19T13:36:39Z

+| `targetFileSizeBytes(long)` | Target size for rewritten files | Table property or 512MB | long |
+| `minFileSizeBytes(long)` | Minimum size of files eligible for compaction | 0 | long |
+| `maxFileSizeBytes(long)` | Maximum size of files eligible for compaction | Long.MAX_VALUE | long |
+| `minInputFiles(int)` | Minimum number of files to trigger rewrite | Integer.MAX_VALUE | int |


I think the default value is 5.

Guosmilesmile · 2025-08-19T13:37:11Z

+| `maxFileGroupSizeBytes(long)` | Maximum total size of a file group | 107374182400 (100GB) | long |
+| `maxFilesToRewrite(int)` | Maximum number of files to rewrite per task | Integer.MAX_VALUE | int |
+| `partialProgressEnabled(boolean)` | Enable partial progress commits | false | boolean |
+| `partialProgressMaxCommits(int)` | Maximum commits for partial progress | 10 | int |


Maximum commits for partial progress, if partial progress is enabled

JeonDaehong · 2025-08-19T13:44:39Z

@Guosmilesmile

I reviewed the classes carefully to check the default values, but I got quite a few wrong. My apologies.

For cleanExpiredMetadata, I initially thought it was true, but it seems false is correct, so I’ve updated it accordingly.

I’ve also made other changes based on your feedback.

Thank you for your review.

JeonDaehong · 2025-08-19T13:45:36Z

@pvary

I’ve updated the document according to your feedback.

The Default Lock section was incorrectly added, so I’ve removed it.

Thank you for your review.

Guosmilesmile · 2025-08-19T15:05:23Z

+        );
+
+        // Set up maintenance with comprehensive configuration
+        TableMaintenance.forTable(env, tableLoader, TriggerLockFactory.defaultLockFactory())


TriggerLockFactory.defaultLockFactory() We don't have this method. This part should be changed.

Guosmilesmile · 2025-08-19T15:09:28Z

+```
+
+#### ZooKeeper Lock Factory
+Uses Apache ZooKeeper for distributed coordination:


Suggested change

Uses Apache ZooKeeper for distributed coordination:

Uses Apache ZooKeeper for distributed locks:

Guosmilesmile · 2025-08-19T15:16:54Z

+| Method | Description | Default Value | Type |
+|--------|-------------|---------------|------|
+| `targetFileSizeBytes(long)` | Target size for rewritten files | Table property or 512MB | long |
+| `minFileSizeBytes(long)` | Minimum size of files eligible for compaction | 0 | long |


Default value is 75% of target file size

Guosmilesmile · 2025-08-19T15:17:34Z

+|--------|-------------|---------------|------|
+| `targetFileSizeBytes(long)` | Target size for rewritten files | Table property or 512MB | long |
+| `minFileSizeBytes(long)` | Minimum size of files eligible for compaction | 0 | long |
+| `maxFileSizeBytes(long)` | Maximum size of files eligible for compaction | Long.MAX_VALUE | long |


Default value is 180% of target file size

Guosmilesmile · 2025-08-19T15:18:03Z

+| `targetFileSizeBytes(long)` | Target size for rewritten files | Table property or 512MB | long |
+| `minFileSizeBytes(long)` | Minimum size of files eligible for compaction | 0 | long |
+| `maxFileSizeBytes(long)` | Maximum size of files eligible for compaction | Long.MAX_VALUE | long |
+| `minInputFiles(int)` | Minimum number of files to trigger rewrite | 1 | int |


Default value is 5

Guosmilesmile · 2025-08-19T15:24:52Z

+| `deleteFileThreshold(int)` | Minimum delete-file count per data file to force rewrite | Integer.MAX_VALUE | int |
+| `rewriteAll(boolean)` | Rewrite all data files regardless of thresholds | false | boolean |
+| `maxFileGroupSizeBytes(long)` | Maximum total size of a file group | 107374182400 (100GB) | long |
+| `maxFilesToRewrite(int)` | Maximum number of files to rewrite per task | Integer.MAX_VALUE | int |


Default value is null. About the description we can add If this option is not specified, all eligible files will be rewritten.

Guosmilesmile · 2025-08-19T15:34:41Z

+  'namespace' = 'db',
+  'table' = 'tbl',


Key should be catalog-database and catalog-table

Guosmilesmile · 2025-08-19T15:40:36Z

+    jdbcProps                                   // JDBC connection properties
+);
+```
+


Can we add some specific configuration item explanations, such as INIT_LOCK_TABLES_PROPERTY, since this configuration item is mentioned in the SQL part below but does not appear in the Java part?

iceberg/flink/v2.0/flink/src/main/java/org/apache/iceberg/flink/maintenance/api/JdbcLockFactory.java

Line 48 in 0478ff7

"flink-maintenance.lock.jdbc.init-lock-tables";

Guosmilesmile · 2025-08-19T15:46:23Z

+- Adjust `deleteBatchSize` based on storage performance
+- Enable `partialProgressEnabled` for large rewrite operations
+- Set reasonable `maxRewriteBytes` limits
+


May be we can add Setting an appropriate maxFileGroupSizeBytes can break down large FileGroups into smaller ones, thereby increasing the speed of parallel processing.

Guosmilesmile · 2025-08-19T15:50:21Z

+| `partialProgressEnabled(boolean)` | Enable partial progress commits | false | boolean |
+| `partialProgressMaxCommits(int)` | Maximum commits allowed for partial progress when partialProgressEnabled is true | 10 | int |
+| `maxRewriteBytes(long)` | Maximum bytes to rewrite per execution | Long.MAX_VALUE | long |
+


Miss a method filter

JeonDaehong · 2025-08-21T23:58:46Z

@pvary @Guosmilesmile

Hello, I’ve incorporated the feedback you mentioned above.

This is my first time writing a Docs from start to finish, so while I’m enjoying it, I also feel like I might be causing some trouble, haha...

That said, please continue to provide feedback. I really appreciate it! :D

Guosmilesmile

I have left some mirror comments. I think we can reuse the existing page https://iceberg.apache.org/docs/nightly/flink-actions/ instead of adding a new file. We can rename flink-actions to flink-maintenance, which would eliminate the need for a new page.

Guosmilesmile · 2025-08-24T10:53:58Z

 | `partialProgressEnabled(boolean)` | Enable partial progress commits | false | boolean |
 | `partialProgressMaxCommits(int)` | Maximum commits allowed for partial progress when partialProgressEnabled is true | 10 | int |
 | `maxRewriteBytes(long)` | Maximum bytes to rewrite per execution | Long.MAX_VALUE | long |
+| `filter(Expression)` | Filter expression for selecting files to rewrite | null | Expression |


Default value is Expressions.alwaysTrue()

Guosmilesmile · 2025-08-24T10:55:35Z

 | `rewriteAll(boolean)` | Rewrite all data files regardless of thresholds | false | boolean |
 | `maxFileGroupSizeBytes(long)` | Maximum total size of a file group | 107374182400 (100GB) | long |
-| `maxFilesToRewrite(int)` | Maximum number of files to rewrite per task | Integer.MAX_VALUE | int |
+| `maxFilesToRewrite(int)` | About the description we can add: If this option is not specified, all eligible files will be rewritten | null | int |


We don't need About the description we can add: in the doc.

JeonDaehong · 2025-08-24T11:09:23Z

I have left some mirror comments. I think we can reuse the existing page https://iceberg.apache.org/docs/nightly/flink-actions/ instead of adding a new file. We can rename flink-actions to flink-maintenance, which would eliminate the need for a new page.

Thank you for the feedback. Once I revise the parts you mentioned and add them to the Flink action docs, would that be all that’s needed?

Guosmilesmile · 2025-08-24T11:19:00Z

I have left some mirror comments. I think we can reuse the existing page https://iceberg.apache.org/docs/nightly/flink-actions/ instead of adding a new file. We can rename flink-actions to flink-maintenance, which would eliminate the need for a new page.

Thank you for the feedback. Once I revise the parts you mentioned and add them to the Flink action docs, would that be all that’s needed?

I don’t see any other changes needed for now. Once you finish the revisions, I will review it again. Thanks for your hard work!

JeonDaehong · 2025-08-24T11:29:20Z

@Guosmilesmile

Thank you. I’ll reflect the points you mentioned, add them to the Flink action, and push the changes again. Thanks.

JeonDaehong · 2025-08-24T13:46:30Z

@Guosmilesmile

How should we handle the 'Rewrite Files' action in flink-action?

JeonDaehong · 2025-08-24T14:14:43Z

@Guosmilesmile

I have first merged the content into flink-action and deleted the original maintenance md file.

Additionally, I have incorporated the two feedback points you mentioned.

If there are no issues, please proceed with the merge.

Thank you.

Guosmilesmile

flink-configuration.mdshould not be within the scope of the current changes. Could it have been accidentally deleted?
May be we should also change the title: "Flink Actions" in flink-actions.md to title: "Flink TableMaintenance", and rename flink-actions.md to flink-maintenance.md , then change the mkdocs.yml ?

Guosmilesmile · 2025-08-24T15:39:21Z

 -->

-## Rewrite files action
+# Rewrite files action


Suggested change

# Rewrite files action

## Flink Table Maintenance BatchMode

### Rewrite files action

Guosmilesmile · 2025-08-24T15:41:04Z


 For more details of the rewrite files action, please refer to [RewriteDataFilesAction](../../javadoc/{{ icebergVersion }}/org/apache/iceberg/flink/actions/RewriteDataFilesAction.html)
+
+# Flink Table Maintenance 


Suggested change

# Flink Table Maintenance

## Flink Table Maintenance StreamingMode

Guosmilesmile · 2025-08-24T15:42:19Z

+
+# Flink Table Maintenance 
+
+## Overview


Suggested change

## Overview

### Overview

Guosmilesmile · 2025-08-24T15:42:33Z

+
+The `TableMaintenance` API in **Apache Iceberg** empowers **Flink jobs** to execute maintenance tasks **natively**, either embedded within existing streaming pipelines or deployed as standalone Flink jobs. This eliminates dependencies on external systems, thereby **streamlining architecture**, **reducing operational costs**, and **enhancing automation capabilities**.
+
+## Supported Features (Flink)


Suggested change

## Supported Features (Flink)

### Supported Features (Flink)

Guosmilesmile · 2025-08-24T15:42:42Z

+
+## Supported Features (Flink)
+
+### ExpireSnapshots


Suggested change

### ExpireSnapshots

#### ExpireSnapshots

Guosmilesmile · 2025-08-24T15:46:15Z

+- Use `scheduleOnCommitCount` for write-heavy tables
+- Use `scheduleOnDataFileCount` for fine-grained control
+
+### Performance Tuning


Suggested change

### Performance Tuning

#### Performance Tuning

Guosmilesmile · 2025-08-24T15:46:22Z

+- Set reasonable `maxRewriteBytes` limits
+- Setting an appropriate `maxFileGroupSizeBytes` can break down large FileGroups into smaller ones, thereby increasing the speed of parallel processing
+
+## Troubleshooting


Suggested change

## Troubleshooting

### Troubleshooting

Guosmilesmile · 2025-08-24T15:46:30Z

+
+## Troubleshooting
+
+### OutOfMemoryError during file deletion


Suggested change

### OutOfMemoryError during file deletion

#### OutOfMemoryError during file deletion

Guosmilesmile · 2025-08-24T15:46:36Z

+.deleteBatchSize(500) // Example: 500 files per batch
+```
+
+### Lock conflicts


Suggested change

### Lock conflicts

#### Lock conflicts

Guosmilesmile · 2025-08-24T15:46:43Z

+.rateLimit(Duration.ofMinutes(10))     // Reduce frequency of task execution
+```
+
+### Slow rewrite operations


Suggested change

### Slow rewrite operations

#### Slow rewrite operations

JeonDaehong · 2025-08-24T22:24:08Z

flink-configuration.mdshould not be within the scope of the current changes. Could it have been accidentally deleted?

May be we should also change the title: "Flink Actions" in flink-actions.md to title: "Flink TableMaintenance", and rename flink-actions.md to flink-maintenance.md , then change the mkdocs.yml ?

@Guosmilesmile
It looks like I mistakenly deleted Maintenance when I should have just removed it. I’ll restore it.

I’ll make the corrections you mentioned today and push the update again.

Thank you for the feedback!

Guosmilesmile · 2025-08-25T02:08:51Z

@JeonDaehong I didn't see the relevant changes. Have you submitted them?

JeonDaehong · 2025-08-25T02:15:41Z

@JeonDaehong I didn't see the relevant changes. Have you submitted them?

@Guosmilesmile

Not yet!
I’m at the office right now.
I plan to proceed after work.

It’s currently 11 AM KST, so I should be able to start after 7 PM, about 8 hours from now.

Would that be alright?

Or, if it’s urgent, I can make the changes and push them even during lunch.

Guosmilesmile · 2025-08-25T04:38:39Z

@JeonDaehong Sorry, I thought you had already submitted the changes when I saw your reply. Just take your time.

JeonDaehong · 2025-08-25T04:40:30Z

@JeonDaehong Sorry, I thought you had already submitted the changes when I saw your reply. Just take your time.

It seems the Korean → English translation didn’t come out quite right. Haha..

…pdate mkdocs.yml

JeonDaehong · 2025-08-25T13:00:18Z

@Guosmilesmile

I have updated the content as per your feedback!

Restored the mistakenly deleted flink-configuration.md file
Renamed and merged flink-action.md into flink-maintenance.md
Updated the content inside flink-maintenance.md as suggested

Please review and merge if everything looks good.

Thank you!

pvary · 2025-08-25T14:02:41Z

+```java
+import org.apache.iceberg.flink.actions.Actions;
+
+TableLoader tableLoader = TableLoader.fromHadoopTable("hdfs://nn:8020/warehouse/path");


fromHadoopTable -> Can we change this to hive or something else?
We would like to remove the references of the HadoopTable from the docs as it is not encouraged in production environments

pvary · 2025-08-25T14:02:50Z

+
+```java
+StreamExecutionEnvironment env = StreamExecutionEnvironment.getExecutionEnvironment();
+TableLoader tableLoader = TableLoader.fromHadoopTable("path/to/table");


fromHadoopTable -> Can we change this to hive or something else?
We would like to remove the references of the HadoopTable from the docs as it is not encouraged in production environments

pvary

2 small changes and then we are good to go.
Thanks @JeonDaehong for the PR and @Guosmilesmile for the review!

JeonDaehong · 2025-08-25T14:08:32Z

@pvary

Is it right to change Hadoop to Hive like this?

pvary · 2025-08-25T14:17:28Z

+import org.apache.iceberg.flink.actions.Actions;
+
+TableLoader tableLoader = TableLoader.fromCatalog(
+    CatalogLoader.hive("my_catalog", configuration),


CatalogLoader.hive("my_catalog", configuration, properties),

I have just completed adding it.

pvary · 2025-08-25T14:28:46Z

Merged to main.
Thanks for the doc @JeonDaehong and @Guosmilesmile for the review!

JeonDaehong · 2025-08-25T14:31:14Z

@pvary @Guosmilesmile

Thank you very much for reviewing and merging!
I will continue to work hard to contribute as a committer and help make Iceberg even better. :D

JeonDaehong · 2025-08-25T14:33:33Z

@pvary

Has this Docs been visible on the Docs website since the 1.9.3 release?

pvary · 2025-08-25T15:00:52Z

It is available here: https://iceberg.apache.org/docs/nightly/flink-maintenance/

JeonDaehong · 2025-08-25T22:05:54Z

It is available here: https://iceberg.apache.org/docs/nightly/flink-maintenance/

Thanks :D

Docs: Add docs for Table Maintenance in Flink

f93d4ae

github-actions Bot added the docs label Aug 18, 2025

pvary reviewed Aug 18, 2025

View reviewed changes

Comment thread docs/docs/flink-table-maintenance.md Resolved

docs : Update MD file incorporating feedback

04f1f34

pvary reviewed Aug 19, 2025

View reviewed changes

Comment thread docs/docs/flink-table-maintenance.md Resolved

pvary reviewed Aug 19, 2025

View reviewed changes

Comment thread docs/docs/flink-table-maintenance.md Resolved

pvary reviewed Aug 19, 2025

View reviewed changes

Comment thread docs/docs/flink-table-maintenance.md Resolved

Guosmilesmile reviewed Aug 19, 2025

View reviewed changes

JeonDaehong added 2 commits August 19, 2025 22:38

docs : Update MD file incorporating feedback

32509e5

docs : Update MD file incorporating feedback

4faa0f5

docs : Update MD file incorporating feedback

e6f11e3

Guosmilesmile reviewed Aug 19, 2025

View reviewed changes

JeonDaehong and others added 2 commits August 21, 2025 21:09

Merge branch 'apache:main' into issue_main

ce6f5f1

Reflected maintainer's feedback

d3adb65

Guosmilesmile reviewed Aug 24, 2025

View reviewed changes

Merge branch 'apache:main' into issue_main

77aad01

docs: merge Flink Maintenance and Flink Action docs and apply feedback

d132084

Guosmilesmile reviewed Aug 24, 2025

View reviewed changes

apply feedback: rename flink-actions.md to flink-maintenance.md and u…

3bf6222

…pdate mkdocs.yml

Guosmilesmile approved these changes Aug 25, 2025

View reviewed changes

pvary reviewed Aug 25, 2025

View reviewed changes

pvary approved these changes Aug 25, 2025

View reviewed changes

Apply minor review feedback

07c4474

pvary reviewed Aug 25, 2025

View reviewed changes

Apply minor review feedback

c2d6f47

pvary merged commit 2df7dc7 into apache:main Aug 25, 2025


		## Overview

		In Apache Iceberg deployments within Flink streaming environments, implementing automated table maintenance operations—including `snapshot expiration`, `small file compaction`, and `orphan file cleanup`—is critical for optimal performance and storage efficiency.

	Uses Apache ZooKeeper for distributed coordination:
	Uses Apache ZooKeeper for distributed locks:

	# Rewrite files action
	## Flink Table Maintenance BatchMode
	### Rewrite files action


		For more details of the rewrite files action, please refer to [RewriteDataFilesAction](../../javadoc/{{ icebergVersion }}/org/apache/iceberg/flink/actions/RewriteDataFilesAction.html)

		# Flink Table Maintenance

	# Flink Table Maintenance
	## Flink Table Maintenance StreamingMode


		The `TableMaintenance` API in Apache Iceberg empowers Flink jobs to execute maintenance tasks natively, either embedded within existing streaming pipelines or deployed as standalone Flink jobs. This eliminates dependencies on external systems, thereby streamlining architecture, reducing operational costs, and enhancing automation capabilities.

		## Supported Features (Flink)

	### OutOfMemoryError during file deletion
	#### OutOfMemoryError during file deletion

Conversation

JeonDaehong commented Aug 18, 2025

Uh oh!

JeonDaehong commented Aug 18, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

JeonDaehong commented Aug 19, 2025

Uh oh!

JeonDaehong commented Aug 19, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

JeonDaehong commented Aug 19, 2025

Uh oh!

JeonDaehong commented Aug 19, 2025

Uh oh!

Guosmilesmile Aug 19, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

JeonDaehong commented Aug 21, 2025

Uh oh!

Guosmilesmile left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

JeonDaehong commented Aug 24, 2025

Uh oh!

Guosmilesmile commented Aug 24, 2025

Uh oh!

JeonDaehong commented Aug 24, 2025

Uh oh!

JeonDaehong commented Aug 24, 2025

Uh oh!

JeonDaehong commented Aug 24, 2025

Uh oh!

Guosmilesmile left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Guosmilesmile Aug 19, 2025 •

edited

Loading

JeonDaehong commented Aug 25, 2025 •

edited

Loading