seungjoo-choi-bucketplace commented on PR #29149:
URL: https://github.com/apache/flink/pull/29149#issuecomment-5854820894

   Hi @Zakelly, @fredia and @Myasuka, could you take a look at this when you 
have a chance?
   
   It fixes TTL background cleanup being silently skipped for State V2 (async) 
states on ForSt: the compaction filter factory is attached to the column 
family, but the TTL configuration is never pushed into it on the V2 path, so 
expired entries are never compacted away and the state grows unbounded. The fix 
wires `configCompactFilter` for the V2 `StateDescriptor` path.
   
   It already has a community review and approval from @minxhe, comes with test 
coverage (`ForStTtlCompactFilterTest`), and CI is green — it just needs a 
committer's review to move forward. Given your work on ForSt state 
TTL/compaction, your review would be much appreciated. Thanks!
   


-- 
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]

Reply via email to