Github user sihuazhou commented on the issue:
https://github.com/apache/flink/pull/5705
Hi @tillrohrmann could you please have a look at this? This PR covers multi
bugs that should be addressed...I think maybe stefan is too busy with the
testing works on 1.5 currently...
---
Github user sihuazhou commented on the issue:
https://github.com/apache/flink/pull/5705
I made a fixup for some additional minor bug in
`RocksDBStateBackendConfigTest.java`.
---
Github user sihuazhou commented on the issue:
https://github.com/apache/flink/pull/5705
@StefanRRichter I have updated the code, could you please have a look...
---
Github user StefanRRichter commented on the issue:
https://github.com/apache/flink/pull/5705
You can cover it here and I will review it again ð
---
Github user sihuazhou commented on the issue:
https://github.com/apache/flink/pull/5705
@StefanRRichter Sorry for pinging you again in this PR, but I found there
maybe be another issue for `full checkpoint`. It's about the
`kvStateInformation`, we should create a deep copy of `kvState
Github user StefanRRichter commented on the issue:
https://github.com/apache/flink/pull/5705
LGTM ð
---
Github user sihuazhou commented on the issue:
https://github.com/apache/flink/pull/5705
@StefanRRichter Could you please have a look at this?
---