[GitHub] [flink] fredia commented on a diff in pull request #20421: [FLINK-28732][state] Deprecate ambiguous StateTTLConfig#cleanFullSnapshot API

[GitBox <gi...@apache.org>]

fredia commented on code in PR #20421:
URL: https://github.com/apache/flink/pull/20421#discussion_r941966732


##########
docs/content/docs/dev/datastream/fault-tolerance/state.md:
##########
@@ -474,9 +474,10 @@ ttl_config = StateTtlConfig \
 For more fine-grained control over some special cleanup in background, you can configure it separately as described below.
 Currently, heap state backend relies on incremental cleanup and RocksDB backend uses compaction filter for background cleanup.
 
-##### Cleanup in full snapshot
+##### Cleanup in full scan snapshot
 
-Additionally, you can activate the cleanup at the moment of taking the full state snapshot which 
+Additionally, you can activate the cleanup at the moment of taking the full scan state snapshot (means the 
+canonical savepoint of all state-backends, or the checkpoint/native-savepoint of hashmap state-backend.), which 

Review Comment:
   Will RocksDB non-incremental checkpoints support `cleanupInRocksdbCompactFilter` After FLINK-28699？



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org