rkhachatryan commented on code in PR #20421: URL: https://github.com/apache/flink/pull/20421#discussion_r941260334
########## 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 Review Comment: I mean that **just** enumarating the cases is enough to clarify this TTL policy; But it's not enough to explain what the "full scan snapshot" is. So I think either the definition of "full scan snapshot" should be given; or the "full scan snapshot" term should be removed. Removing seems preferrable lest overload users with unnecessary information. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
