zentol commented on a change in pull request #10866: Enable TTL compaction filter by default and deprecate/remove some other TTL API URL: https://github.com/apache/flink/pull/10866#discussion_r367058943
########## File path: flink-state-backends/flink-statebackend-rocksdb/src/main/java/org/apache/flink/contrib/streaming/state/RocksDBOptions.java ########## @@ -59,10 +59,15 @@ .defaultValue(1) .withDescription("The number of threads (per stateful operator) used to transfer (download and upload) files in RocksDBStateBackend."); - /** This determines if compaction filter to cleanup state with TTL is enabled. */ + /** + * This determines if compaction filter to cleanup state with TTL is enabled. + * + * @deprecated the option will be removed in future and have no effect. Review comment: `should only be used >when< experiencing serious ...` ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services