Zakelly
Zakelly
@jihuayu Sure thing. Log attached. [logs_21881009491.zip](https://github.com/apache/kvrocks/files/14734551/logs_21881009491.zip)
Happy to see the great investigation! Thanks @AntiTopQuark
Hi @masteryhx , sorry for the late reply! After reading your PR, I finally figured out why the default implementation of old method and new method should call each other....
Sorry for jumping in but I have mixed feelings... Is this a common problem that also applies for other options and shall we implement a generic solution for all similar...
Sorry for jumping in but may I ask about the current status?
Thanks for the PR! I suggest creating a jira ticket for this. And it would be better if the description in `org.apache.flink.configuration.RestartStrategyOptions` is also fixed.
@fredia @masteryhx Would you please help review this?
@fredia @masteryhx Thanks for your review! I've addressed your comments PTAL.
> Thanks for the update. Overall LGTM. Just minor suggestions: > > 1. Update related docs from 'Restore Mode' to 'Claim Mode' to provide clearer guidance for users. > 2....
> > Hi @spoon-lz , thanks for your contribution! At first glance of your PR, I would suggest keep the old `org.apache.flink.streaming.api.CheckpointingMode` and mark as deprecated. However it is required...