apache
[FLINK-37294][state] Support state migration between disabling and enabling ttl in HeapKeyedStateBackend
[FLINK-37294][state] Support state migration between disabling and enabling ttl in HeapKeyedStateBackend
What is the purpose of the change
Support state migration between disabling and enabling ttl in HeapKeyedStateBackend
Brief change log
Add migrateTtlValue in AbstractHeapState. When the state TTL switch changes, trigger the migration of state data.
Verifying this change
This change is already covered by existing tests, such as StateBackendMigrationTestBase#testStateMigrationAfterChangingTTLFromEnablingToDisabling and StateBackendMigrationTestBase#testStateMigrationAfterChangingTTLFromDisablingToEnabling.
Does this pull request potentially affect one of the following parts:
- Dependencies (does it add or upgrade a dependency): no
- The public API, i.e., is any changed class annotated with
@Public(Evolving): no - The serializers: no
- The runtime per-record code paths (performance sensitive): no
- Anything that affects deployment or recovery: no
- The S3 file system connector: no
Documentation
- Does this pull request introduce a new feature? yes
- If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
CI report:
- 82ea4f008db802639a061137c0045effff49b3fc Azure: SUCCESS
Bot commands
The @flinkbot bot supports the following commands:@flinkbot run azurere-run the last Azure build
@hejufang Hi, I did some improvement for your implementation in this PR(https://github.com/apache/flink/pull/26674). I've also changed this commit as a co-authored commit. PTAL.
@hejufang Hi, I did some improvement for your implementation in this PR(#26674). I've also changed this commit as a co-authored commit. PTAL.
@xiangyuf Thank you for the improvements. I have push the new commit to the current branch, and we can continue to track this PR.
