echauchot commented on PR #21636: URL: https://github.com/apache/flink/pull/21636#issuecomment-1457760168
> What you did for versioning looks sane. I think the problem is with line > > https://github.com/apache/flink/blob/a4a11a389609deb5d1de139115cedc35b675b366/flink-runtime/src/main/java/org/apache/flink/runtime/state/metainfo/StateMetaInfoSnapshotReadersWriters.java#L68 > > which has not been implemented in a future proof way. The intent there was to throw an exception for `readVersion < 6`, not less than `CURRENT_STATE_META_INFO_SNAPSHOT_VERSION`. If we change the condition to `readVersion < 6` it should work. Hi @dawidwys, thanks for the prompt answer. I suspected something not future proof, thanks for the confirmation. It is now fixed. Tests pass but the CI fails in preparation of the E2E tests (since some days) with a http 404 error. PTAL. -- 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: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
